WinFF / winff

Automatically exported from code.google.com/p/winff
214 stars 50 forks source link

Wish-list: Ability to choose whether a crop is applied to input or output #77

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago

What is the expected output? What do you see instead?
Use of the crop feature in "Additional Options" only allows crops to be
applied to the output video. FFmpeg also allows crops to be applied to the
input video before it is re-sized.
It makes a difference whether a crop is applied to the input or output.
Crops applied to the input cut out unwanted image content and then resize
to match the specified output dimensions. Crops applied to the output occur
after the image is resized and results in the output file's dimensions
being smaller than the specified dimensions.

What version of the product are you using? On what operating system?
1.2 on Windows Vista

Please provide any additional information below.
Whether you add crop parameters by the "Additional Command Line Parameters"
field or by the crop tab, WinFF adds parameters near the end of the command
line. This instructs FFmpeg to perform the crop to the output. After some
experimenting, I discovered that FFmpeg will apply a crop to the input if
the crop parameters are inserted before the output size is specified. I
have presets configured to fit the dimension of a mobile device's screen.
When I have a video that is too wide to fit and I want to crop the edges,
the crop feature performs incorrectly. It stretches the widescreen images
out of proportion and produces a file with smaller dimensions than my
device's screen. A perfect example of why this feature is needed. I am not
sure, but I believe the same feature could be added for a pad function as well.

Original issue reported on code.google.com by QBi...@gmail.com on 27 May 2010 at 1:38

GoogleCodeExporter commented 9 years ago

Original comment by poipodec...@hotmail.com on 27 May 2010 at 5:28

GoogleCodeExporter commented 9 years ago
I have committed the fix to svn.  Please check.

Original comment by istoff@gmail.com on 1 Jun 2010 at 7:01

GoogleCodeExporter commented 9 years ago

Original comment by istoff@gmail.com on 13 Jul 2010 at 6:54