On Wed, Nov 02, 2016 at 20:39 +0100, Gerhard Sittig wrote:
> 
> Hi!
> 
> When I looked at several places in the sigrok's subprojects'
> sources, I noticed the absence of common endianess conversion
> helpers.  All affected call sites appear to open code the
> respective functions.

This was written from the top of my head.  Apparently I was
exaggerating. :)

Doing the recursive search for WORDS_BIGENDIAN across several
repos, one can see that strictly only libsigrok is affected, and
it's just a few spots: analog data handling, tests for analog
data, WAV input and output modules, Saleae logic analyzer driver,
C++ binding.

These are the spots which reference the endianess as it was
detected by autoconf.  Haven't checked how much further the
effect goes that depends on derived information (stored condition
variables which later influence other code paths).  And haven't
looked for locations which handle endianess but don't reference
the WORDS_BIGENDIAN condition.

Still it might be worth to factor out the endianess handling.
Asix Sigma (WIP) is a current motivation.  Other locations could
benefit from common logic.


virtually yours
Gerhard Sittig
-- 
     If you don't understand or are scared by any of the above
             ask your parents or an adult to help you.

------------------------------------------------------------------------------
Developer Access Program for Intel Xeon Phi Processors
Access to Intel Xeon Phi processor-based developer platforms.
With one year of Intel Parallel Studio XE.
Training and support from Colfax.
Order your platform today. http://sdm.link/xeonphi
_______________________________________________
sigrok-devel mailing list
sigrok-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/sigrok-devel

Reply via email to