[FFmpeg-devel] [PATCH 1/5] Add av_ prefix to bswap macros
Måns Rullgård
mans
Sat Jul 10 22:43:38 CEST 2010
Michael Niedermayer <michaelni at gmx.at> writes:
> On Sat, Jul 10, 2010 at 10:33:18PM +0200, Michael Niedermayer wrote:
>> On Sat, Jul 10, 2010 at 01:22:31PM +0100, Mans Rullgard wrote:
>> [...]
>> > -// be2me ... big-endian to machine-endian
>> > -// le2me ... little-endian to machine-endian
>> > +// be2ne ... big-endian to native-endian
>> > +// le2ne ... little-endian to native-endian
>> >
>> > #if HAVE_BIGENDIAN
>> > -#define be2me_16(x) (x)
>> > -#define be2me_32(x) (x)
>> > -#define be2me_64(x) (x)
>> > -#define le2me_16(x) bswap_16(x)
>> > -#define le2me_32(x) bswap_32(x)
>> > -#define le2me_64(x) bswap_64(x)
>> > -#define AV_BE2MEC(s, x) (x)
>> > -#define AV_LE2MEC(s, x) AV_BSWAPC(s, x)
>> > +#define av_be2ne16(x) (x)
>> > +#define av_be2ne32(x) (x)
>> > +#define av_be2ne64(x) (x)
>> > +#define av_le2ne16(x) av_bswap16(x)
>> > +#define av_le2ne32(x) av_bswap32(x)
>> > +#define av_le2ne64(x) av_bswap64(x)
>> > +#define AV_BE2NEC(s, x) (x)
>> > +#define AV_LE2NEC(s, x) AV_BSWAPC(s, x)
>> > #else
>> > -#define be2me_16(x) bswap_16(x)
>> > -#define be2me_32(x) bswap_32(x)
>> > -#define be2me_64(x) bswap_64(x)
>> > -#define le2me_16(x) (x)
>> > -#define le2me_32(x) (x)
>> > -#define le2me_64(x) (x)
>> > -#define AV_BE2MEC(s, x) AV_BSWAPC(s, x)
>> > -#define AV_LE2MEC(s, x) (x)
>> > +#define av_be2ne16(x) av_bswap16(x)
>> > +#define av_be2ne32(x) av_bswap32(x)
>> > +#define av_be2ne64(x) av_bswap64(x)
>> > +#define av_le2ne16(x) (x)
>> > +#define av_le2ne32(x) (x)
>> > +#define av_le2ne64(x) (x)
>> > +#define AV_BE2NEC(s, x) AV_BSWAPC(s, x)
>> > +#define AV_LE2NEC(s, x) (x)
>> > #endif
>> >
>> > -#define AV_BE2ME16C(x) AV_BE2MEC(16, x)
>> > -#define AV_BE2ME32C(x) AV_BE2MEC(32, x)
>> > -#define AV_BE2ME64C(x) AV_BE2MEC(64, x)
>> > -#define AV_LE2ME16C(x) AV_LE2MEC(16, x)
>> > -#define AV_LE2ME32C(x) AV_LE2MEC(32, x)
>> > -#define AV_LE2ME64C(x) AV_LE2MEC(64, x)
>> > +#define AV_BE2NE16C(x) AV_BE2NEC(16, x)
>> > +#define AV_BE2NE32C(x) AV_BE2NEC(32, x)
>> > +#define AV_BE2NE64C(x) AV_BE2NEC(64, x)
>> > +#define AV_LE2NE16C(x) AV_LE2NEC(16, x)
>> > +#define AV_LE2NE32C(x) AV_LE2NEC(32, x)
>> > +#define AV_LE2NE64C(x) AV_LE2NEC(64, x)
>>
>> that is something else ...
>
> i realize iam not the first to spot this ...
I also mentioned it in the overview mail I sent yesterday before
starting on this...
> note iam ok with this change but it should be seperate from adding av_
> i think ideally
OK, then I'll split it.
--
M?ns Rullg?rd
mans at mansr.com
More information about the ffmpeg-devel
mailing list