[FFmpeg-devel] [PATCH] libavfilter-soc: extend vf_scale.c to make it support colorspace details setting

Michael Niedermayer michaelni
Wed May 20 04:02:12 CEST 2009


On Tue, May 19, 2009 at 11:15:05PM +0200, Stefano Sabatini wrote:
> On date Tuesday 2009-05-19 00:52:25 +0200, Michael Niedermayer encoded:
> > On Mon, May 18, 2009 at 10:00:08PM +0200, Stefano Sabatini wrote:
> > > On date Monday 2009-05-18 03:03:00 +0200, Michael Niedermayer encoded:
> > > > On Sun, May 17, 2009 at 01:33:48PM +0200, Stefano Sabatini wrote:
> > > [...]
> > > > > The only alternative I see would be to redefine in the ScaleContext
> > > > > every single option of SWScaleContext, then map them to the
> > > > > corresponding SWScaleContext options when setting them, I avoided it
> > > > > since that looks more complicate.
> > > > > 
> > > > > Would that be acceptable?
> > > > 
> > > > no you seem to be missing the point
> > > > i want, none, ZERO not even a trace of any of these options in the filter
> > > > the filter should pass the stuff generically to swscale one way or another
> > > > This may not be achiveable and compromises might be needed but what your
> > > > patches do goes too far ...
> > > >
> > > > The various ways by which you dupllicate the AVOption list from swscale
> > > > is why iam so unhappy with this patch
> > > >
> > > > I dont know if it can be done without changes to swscale ...
> > > 
> > > I have to confess that at this point I have no idea of what you
> > > mean... you want the scale filter to be configurable but
> > > "generically", which I don't know at all how to interpret.
> > 
> > heres a example:
> > 
> > generic code:
> > for(all input parameters given to us)
> >     give input parameter to sws
> > 
> > 
> > non generic code OTOH checks for "sws_flags" or contains tables of
> > names (if its a little less messy) and does all kinds of obscure
> > things with them
> > 
> > is it clear now?
> 
> My proposal:
> * cpu_caps
> * scaler algo
> * flags (bitexact, accurate_rnd, print_info)
> ...

there are 2 choices
1. cleanup your code
2. suggest changes to swscale and explain why they are needed to
   cleanup your code.

you dont explain what you exactly want to change to what and even less
so why this would reduce the mess in your patch

Now if i guess what you mean, that is not related to why i rejected your
patch.
To oversimplify it a little
Your patch is rejected because instead of passing the letters from the
user to swscale you open them.
Now you seem to suggest to change the content of the letters, how
could that help cleaning your patch?

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Everything should be made as simple as possible, but not simpler.
-- Albert Einstein
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090520/6113d5f2/attachment.pgp>



More information about the ffmpeg-devel mailing list