[ 
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.

Reply via email to