[mythtv] [PATCH] Endian Fixes

Mino Taoyama mythtv at taoyama.com
Sat Jan 1 12:21:17 UTC 2005


Isaac,

I was under the impression that config.h could only be used in 
libavcodec.

I also saw several other areas of the MythTV code that use defines that 
are set by the platform/architecture include files and thought that an 
automated detection mechanism would be a clean solution rather than 
having to hand edit the settings.pro file or even adding a config.h 
include.

I'll pull my detection changes out of the patch and resubmit the audio 
endian fixes to use WORDS_BIGENDIAN.

For future reference can you clarify if using config.h in code other 
than libavcodec is OK?

Thanks,
Mino


On Dec 31, 2004, at 10:09 PM, Isaac Richards wrote:

> On Friday 31 December 2004 05:16 am, Mino Taoyama wrote:
>> Enclosed is a patch against a 12/30/04 cvs checkout that adds the 
>> audio
>> fixes for big endian machines. Thanks Nigel! It replace the
>> WORDS_BIGENDIAN define with automatic endian detections.
>
> Why?  WORDS_BIGENDIAN is automatically defined in config.h.  Files 
> that need
> that should include it.
>
> This won't be applied as is.
>
> Isaac
> _______________________________________________
> mythtv-dev mailing list
> mythtv-dev at mythtv.org
> http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-dev
>



More information about the mythtv-dev mailing list