[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