[FFmpeg-devel] [PATCH] mov auto crop

Baptiste Coudurier baptiste.coudurier
Sat Oct 2 22:03:28 CEST 2010


On 10/2/10 5:57 AM, Benjamin Larsson wrote:
> On 10/02/2010 04:51 AM, Baptiste Coudurier wrote:
>> Hi guys,
>>
>> $subject.
>>
>> Please keep in mind that this issue has been present for _years_ and
>> nobody dared to fix it. I believe the patch is not so hackish/ugly.
>>
>> I restrict it to codecs not supporting arbitrary resolutions that can be
>> stored in .mov by quicktime.
>>
>> Maybe exporting to "crop", "wxh" is better, I don't know.
>>
>> I plan to add auto-rotate for iphone 3gs files this way as well :)
>>
> 
> Jolly good. But would it be possible to move this code to the api and
> just not in the ffmpeg commandline tool? Ie so the users of the api also
> would get these fixes. Other then that I like the idea.

I'm not sure I understand. You need to crop after decoding, or you mean
cropping in the decoder ?

-- 
Baptiste COUDURIER
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
FFmpeg maintainer                                  http://www.ffmpeg.org



More information about the ffmpeg-devel mailing list