Mailing-List: contact dev-help@activemq.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@activemq.apache.org
Message-ID: <14280679.1182801093208.JavaMail.jira@brutus>
Date: Mon, 25 Jun 2007 12:51:33 -0700 (PDT)
From: "Gerald Kaas (JIRA)" <jira@apache.org>
To: dev@activemq.apache.org
Subject: [jira] Commented: (AMQCPP-93) Performance analysis
In-Reply-To: <20942071.1174226494454.JavaMail.jira@brutus>
MIME-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit


    [ https://issues.apache.org/activemq/browse/AMQCPP-93?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_39517 ] 

Gerald Kaas commented on AMQCPP-93:
-----------------------------------

I'm not sure why you want to get rid of the Endian class. The WORDS_BIGENDIAN macro can be replaced with...

  const int t1 = 1;
  const char* const t2 = (const char* const) &a;
  if (t1 != *t2)
  {
  ....

All modern compilers will optimize this to choose either one branch or the other. But, it probably would make sense to move the code from Endian.cpp to Endian.h. That way the compiler can inline the code at compile time rather than link time.

> Performance analysis
> --------------------
>
>                 Key: AMQCPP-93
>                 URL: https://issues.apache.org/activemq/browse/AMQCPP-93
>             Project: ActiveMQ C++ Client
>          Issue Type: Task
>    Affects Versions: 2.0
>            Reporter: Nathan Mittler
>            Assignee: Nathan Mittler
>             Fix For: 2.2
>
>         Attachments: amqcpp-perf1.patch, amqcpp-perf1v2.patch, bench1.cpp, bench2.cpp, DataInputStream.patch, DataOutputStreamv0.patch, patch Results.xls
>
>
> Do a performance analysis on openwire vs stomp.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.