On Fri, Jan 13, 2012 at 09:05:37AM -0800, Ronald S. Bultje wrote:
> On Fri, Jan 13, 2012 at 8:01 AM, Diego Biurrun <[email protected]> wrote:
> > ---
> >  libswscale/colorspace-test.c |    2 +-
> >  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> Why?

I was trawling -Wextra output for sensible issues, this was one that
turned up.

> Somewhat related, I'd prefer if we used them. I wanted to work on that
> but forgot. E.g., we could think of using argv[1]/[2] to specify a
> force-input or force-output format (or both), so we can test specific
> conversions if we want to work on that. Also, being able to specify an
> input and output file (for source data) so we can actually fill the
> buffer with meaningful data, and perhaps store the result to be
> inspected later (all raw of course) would be incredibly useful.

By all means, make it useful and put it back when the time comes.
Until then, this is just pointless clutter and should be removed.

Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to