[FFmpeg-devel] [PATCH 2/2] dsputil/me/pixels: Actually use av_restrict

Ronald S. Bultje rsbultje at gmail.com
Sat Sep 8 22:43:42 CEST 2012


Hi,

On Sat, Sep 8, 2012 at 2:35 PM, Derek Buitenhuis
<derek.buitenhuis at gmail.com> wrote:
> On 07/09/2012 4:19 PM, Michael Niedermayer wrote:
>> i wonder, isnt it simpler and cleaner if these get replaced by that
>> msvc wraper script ?
>> or was there some other platform/case that needs these ?
>
> I think there is a tentative patch for this in the converter somewhere. I think it
> is intended to make it in.

Well, tbh my intention was to keep this kind of stuff out; I'd prefer
if the converter only did that: convert c99 code to c89 code, or more
specifically, convert designated initializers and compound literals to
c89-compatible forms of the same code. I believe this adds to
simplicity and maintainability of the whole thing.

If there's really no other way, we can consider putting this in also,
but I'd really prefer if the script didn't become a "oh it's
msvc-related, let's throw it in there" bottomless pit of crap.

Ronald


More information about the ffmpeg-devel mailing list