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

Måns Rullgård mans
Fri Oct 24 04:15:51 CEST 2008


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

> On Thu, Oct 23, 2008 at 5:52 PM, M?ns Rullg?rd <mans at mansr.com> wrote:
>> "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.
>
> I was referring to the "Please educate the glib guys about this" comment.

That comment is still valid.

-- 
M?ns Rullg?rd
mans at mansr.com




More information about the ffmpeg-devel mailing list