Thanks for feedbacks.
I fixed my source.
Please see it.
2008/5/11 Vitor Sessak <[EMAIL PROTECTED]>:
>> + 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/
diff.patch
Description: Binary data
_______________________________________________ FFmpeg-soc mailing list [email protected] https://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-soc
