On 9/13/2017 3:36 PM, Hendrik Leppkes wrote: > On Wed, Sep 13, 2017 at 7:49 PM, James Almer <jamr...@gmail.com> wrote: >> On 9/13/2017 2:23 PM, Carl Eugen Hoyos wrote: >>> 2017-09-13 19:10 GMT+02:00 James Almer <jamr...@gmail.com>: >>>> Some targets, like NetBSD and DJGPP, don't seem to support it. >>>> >>>> Signed-off-by: James Almer <jamr...@gmail.com> >>>> --- >>>> 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 > ffmpeg-devel@ffmpeg.org > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel > _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
