[FFmpeg-cvslog] libavfilter/libmpcodecs: add vf_stereo3d support

Reimar Döffinger Reimar.Doeffinger at gmx.de
Tue Sep 20 19:29:28 CEST 2011


On Tue, Sep 20, 2011 at 07:11:15PM +0200, Michael Niedermayer wrote:
> On Tue, Sep 20, 2011 at 07:03:26PM +0200, Reimar Döffinger wrote:
> > On Tue, Sep 20, 2011 at 06:19:14AM +0200, Michael Niedermayer wrote:
> > > parameters must be integers as we do not have mplayers parser
> > 
> > Well, it would just be a matter of trawling through the table to also
> > support the strings.
> > Or maybe simpler, only support the strings.
> 
> when i added the libmpcodec wraper my attempt at more or less full
> support of the parser failed because it nearly depends on every
> single file in mplayers tree :)

I wouldn't have recommend that anyway, honestly using the parser here
is a whole lot more code than just parsing it manually.
I was only trying to suggest that since the table is already in a
suitable format parsing the strings is not much more effort than only
parsing the ints.

> > > also the stuff passed to it through non constant globals is not supported
> > > this should be fixed in mplayer first.
> > 
> > Mentioning which exactly those are would increase the likelyhood
> > if it being fixed...
> 
> sorry, didnt think of that
> its opt_screen_size_x && opt_screen_size_y

Ok, that's not really a feature but a hack.
The filter will by default adjust the aspect (by adjusting the final
display size), however if the user explicitly said "I want the output
to be x by y pixels" that obviously shouldn't be done.
I have no idea how, if at all, libavfilter handles this.


More information about the ffmpeg-cvslog mailing list