[FFmpeg-devel] [RFC UNTESTED PATCH] Add IA-64 assembly byteswapping functions.

Måns Rullgård mans
Fri Oct 24 02:52:13 CEST 2008

"Eli Friedman" <eli.friedman at gmail.com> writes:

> On Thu, Oct 23, 2008 at 12:37 AM, Diego Biurrun <diego at biurrun.de> wrote:
>>> +static av_always_inline av_const uint16_t bswap_16(uint16_t x)
>>> +{
>>> +    uint16_t __v;
>> This is an illegal variable name, names starting with __ are reserved
>> for the system.  Please educate the glib guys about this.
> This code is originally from the system header byteswap.h.

But it's not there now.

> glibc can't use any unreserved identifiers there because they could
> be redefined.

While this is true, the origin of the code has no relevance whatsoever
to the symbol names it may use if copied somewhere else.

M?ns Rullg?rd
mans at mansr.com

More information about the ffmpeg-devel mailing list