[FFmpeg-soc] expand filter (alternative to pad syntax)

Ryo Hirafuji psi at science.mi.to
Sun May 11 05:13:47 CEST 2008


Thanks for feedbacks.
I fixed my source.
Please see it.

2008/5/11 Vitor Sessak <vitor1001 at gmail.com>:
>> +     if (expand->ew == -1){
>> +             expand->ew=width;
>> +     } else if (expand->ew < -1){
>> +             expand->ew=width - expand->ew;
>> +     } else if (expand->ew < width){
>> +             expand->ew=width;
>> +     }
>> +
>> +     if (expand->eh == -1){
>> +             expand->eh=height;
>> +     } else if (expand->eh < -1){
>> +             expand->eh=height - expand->eh;
>> +     } else if (expand->eh < height){
>> +             expand->eh=height;
>> +     }
>
> It might be a matter of taste, but I think it would like much better
> without the braces.

This part is compatible with MEncoder's expand filter.
I think it is important to ensure the compatibility with MEncoder's
expand filter.
(In fact, it is because I made a program which uses both ffmpeg and
MEncoder to encode video. ^^;)
So, I think I should not change this.

--
Ryo Hirafuji
blog http://ledyba.ddo.jp/
dojin http://ledyba.ddo.jp/Pegasus/
web http://ledyba.hp.infoseek.co.jp/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: diff.patch
Type: application/octet-stream
Size: 14414 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-soc/attachments/20080511/471e3ecc/attachment-0001.obj>


More information about the FFmpeg-soc mailing list