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 [email protected] http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-dev
_______________________________________________ mythtv-dev mailing list [email protected] http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-dev
