[FFmpeg-devel] [PATCH 1/5] lavc: use LOCAL_ALIGNED instead of DECLARE_ALIGNED

Reimar Döffinger Reimar.Doeffinger at gmx.de
Sat Mar 14 22:54:13 CET 2015


On 14.03.2015, at 20:08, Christophe Gisquet <christophe.gisquet at gmail.com> wrote:

> 2015-03-14 20:06 GMT+01:00 Michael Niedermayer <michaelni at gmx.at>:
>>> So here's an updated patch.
>> 
>> yes, but not in the email ;)
> 
> Crap, I prepared the mail in advance, but forgot to attach the patch
> after the fate run.

I kind of object to making these changes combined.
For the static variables the previous code was simply _correct_ in the alignment macro used.
Now if we think these variables should not be static that is a different point, but that belongs in a different patch with a commit message that correctly describes what the patch actually does.


More information about the ffmpeg-devel mailing list