[FFmpeg-cvslog] v210enc: use FFALIGN()

[Reimar Döffinger]

On 7 Feb 2012, at 08:53, Reimar Döffinger <Reimar.Doeffinger at gmx.de> wrote:
> On 7 Feb 2012, at 02:02, git at videolan.org (Paul B Mahol) wrote:
> 
>> ffmpeg | branch: master | Paul B Mahol <onemda at gmail.com> | Sun Feb  5 21:14:37 2012 +0000| [7ad1b612c8a2a1b1b47f6c3c580ced4bca17e1c7] | committer: Anton Khirnov
>> 
>> v210enc: use FFALIGN()
>> 
>> Signed-off-by: Paul B Mahol <onemda at gmail.com>
>> Signed-off-by: Anton Khirnov <anton at khirnov.net>
>> 
>>> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=7ad1b612c8a2a1b1b47f6c3c580ced4bca17e1c7
>> ---
>> 
>> libavcodec/v210enc.c |    2 +-
>> 1 files changed, 1 insertions(+), 1 deletions(-)
>> 
>> diff --git a/libavcodec/v210enc.c b/libavcodec/v210enc.c
>> index c31254e..00a8902 100644
>> --- a/libavcodec/v210enc.c
>> +++ b/libavcodec/v210enc.c
>> @@ -53,7 +53,7 @@ static int encode_frame(AVCodecContext *avctx, unsigned char *buf,
>>                        int buf_size, void *data)
>> {
>>    const AVFrame *pic = data;
>> -    int aligned_width = ((avctx->width + 47) / 48) * 48;
>> +    int aligned_width = FFALIGN(avctx->width, 48);
> 
> That is not correct I think, FFALIGN only works for powers of 2.

Already fixed, reading emails in order isn't always best.
I suspect that this means we could use a few more tests if this did not break "make fate".