On 11/09/15 10:34, Hendrik Leppkes wrote: > On Fri, Sep 11, 2015 at 5:45 AM, Luca Barbato <[email protected]> wrote: >> On 11/09/15 01:34, Hendrik Leppkes wrote: >>> On Fri, Sep 11, 2015 at 12:44 AM, Luca Barbato <[email protected]> wrote: >>>> On 10/09/15 17:05, Vittorio Giovara wrote: >>>>> From: Luca Barbato <[email protected]> >>>>> >>>>> --- >>>>> libavfilter/graphparser.c | 43 >>>>> ++++++++++++++++++++++++++++++++++++++----- >>>>> 1 file changed, 38 insertions(+), 5 deletions(-) >>>>> >>>> >>>> If it works for you, works for me =) >>> >>> I'm slightly confuzzled. So some time ago you remove the "old" syntax >>> from the generic code, and then you re-add it in filter-specific >>> hacks? >> >> Did I? > > "You" as in Libav. English isn't a very precise language. :p
Back to the issue at hand, that part is under-defined and it had a special-case just for the sws_flags override, that is a special case in itself as well. With the additional syntax dropped from the scale filter, I noticed the override causes problems so I put this not exactly great workaround here. Adding support for mixed positional and named arguments would be nice. Also would be great deprecating the sws-specific syntax and figure out a mean to provide a generic defaults-override for auto-inserted filters. For the time being the patch solves the immediate problem, thought. lu _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
