[FFmpeg-devel] [PATCH] mov auto crop

Michael Niedermayer michaelni
Sat Oct 2 18:20:15 CEST 2010


On Sat, Oct 02, 2010 at 05:58:52PM +0200, Stefano Sabatini wrote:
> On date Saturday 2010-10-02 14:57:43 +0200, Benjamin Larsson encoded:
> > 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.
> 
> An idea could be to provide a callback to the MOV format context,
> which could be initialized by the application and which would instruct
> the application about what to do when such cropping/rotation
> operations are required. But I agree that this looks more bloated and
> provides little gain.

> But maybe we should at least do a check, and
> enable the cropping only if the format is MOV.

well if someone stream copies mov to some other container we would want to
preserve the croping information and use it ideally

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The real ebay dictionary, page 2
"100% positive feedback" - "All either got their money back or didnt complain"
"Best seller ever, very honest" - "Seller refunded buyer after failed scam"
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20101002/cd5c38d5/attachment.pgp>



More information about the ffmpeg-devel mailing list