[FFmpeg-devel] [PATCH 3/3] avcodec/exr: add option to output pixels in float

[Mark Reid]

On Wed, May 6, 2020 at 1:07 PM Anton Khirnov <anton at khirnov.net> wrote:

> Quoting mindmark at gmail.com (2020-04-29 05:02:35)
> > From: Mark Reid <mindmark at gmail.com>
> >
> > ---
> >  libavcodec/exr.c | 103 +++++++++++++++++++++++++++++++++++++++--------
> >  1 file changed, 86 insertions(+), 17 deletions(-)
> >
> > diff --git a/libavcodec/exr.c b/libavcodec/exr.c
> > @@ -1888,6 +1955,8 @@ static const AVOption options[] = {
> >          AV_OPT_TYPE_STRING, { .str = "" }, 0, 0, VD },
> >      { "gamma", "Set the float gamma value when decoding", OFFSET(gamma),
> >          AV_OPT_TYPE_FLOAT, { .dbl = 1.0f }, 0.001, FLT_MAX, VD },
> > +    { "float", "Output frame in floating point pixel format, apply_trc
> does not get applied", OFFSET(output_float),
> > +        AV_OPT_TYPE_BOOL, {.i64 = 0}, 0, 1, VD},
>
> Wouldn't it make more sense to export the coded format rather than have
> an option like this?
>
>
I could do that, I just was worried about breaking any user's use case. But
now that I think about it, I don't think it would, and I think that would
be simpler.


> --
> Anton Khirnov
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".