On 9/13/2017 3:36 PM, Hendrik Leppkes wrote: > On Wed, Sep 13, 2017 at 7:49 PM, James Almer <[email protected]> wrote: >> On 9/13/2017 2:23 PM, Carl Eugen Hoyos wrote: >>> 2017-09-13 19:10 GMT+02:00 James Almer <[email protected]>: >>>> Some targets, like NetBSD and DJGPP, don't seem to support it. >>>> >>>> Signed-off-by: James Almer <[email protected]> >>>> --- >>>> configure | 6 ++++++ >>>> 1 file changed, 6 insertions(+) >>>> >>>> diff --git a/configure b/configure >>>> index ab1550d39b..4855ca54c9 100755 >>>> --- a/configure >>>> +++ b/configure >>>> @@ -3222,6 +3222,7 @@ pixfmts_super2xsai_test_deps="super2xsai_filter" >>>> tinterlace_filter_deps="gpl" >>>> tinterlace_merge_test_deps="tinterlace_filter" >>>> tinterlace_pad_test_deps="tinterlace_filter" >>>> +tonemap_filter_deps="const_nan" >>> >>> No objections but what was wrong with using another >>> default to avoid the additional check? >> >> DBL_MIN and DBL_MAX are both valid values for the option. Turning one of >> them into "auto" does not seem like a good idea to me. >> > > valid perhaps, but also reasonably useful? ie. would anyone ever use > them and get a useful result?
Maybe not, but I'm not going to spend time reading and testing the code to find out if that's the case when the maintainer hasn't dealt with this himself for an entire month. As i said, if anyone has a better or preferred solution I'll not oppose a patch implementing it and reverting mine. > > - Hendrik > _______________________________________________ > ffmpeg-devel mailing list > [email protected] > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel > _______________________________________________ ffmpeg-devel mailing list [email protected] http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
