diff mbox series

[FFmpeg-devel,1/1] avcodec/exr: use lookuptable for alpha if there is no trc_func

Message ID 20201109033723.792-1-mindmark@gmail.com
State Accepted
Commit dd03d6554053ccd4c9bd85429d58c26fe4182de0
Headers show
Series [FFmpeg-devel,1/1] avcodec/exr: use lookuptable for alpha if there is no trc_func | expand

Checks

Context Check Description
andriy/x86_make success Make finished
andriy/x86_make_fate success Make fate finished
andriy/PPC64_make success Make finished
andriy/PPC64_make_fate success Make fate finished

Commit Message

Mark Reid Nov. 9, 2020, 3:37 a.m. UTC 
From: Mark Reid <mindmark@gmail.com>

---
 libavcodec/exr.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Mark Reid Nov. 17, 2020, 3:49 a.m. UTC | #1 
On Sun, Nov 8, 2020 at 7:37 PM <mindmark@gmail.com> wrote:

> From: Mark Reid <mindmark@gmail.com>
>
> ---
>  libavcodec/exr.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libavcodec/exr.c b/libavcodec/exr.c
> index cf7824402a..e907c5c464 100644
> --- a/libavcodec/exr.c
> +++ b/libavcodec/exr.c
> @@ -1203,7 +1203,7 @@ static int decode_block(AVCodecContext *avctx, void
> *tdata,
>                      }
>                  } else if (s->pixel_type == EXR_HALF) {
>                      // 16-bit
> -                    if (c < 3) {
> +                    if (c < 3 || !trc_func) {
>                          for (x = 0; x < xsize; x++) {
>                              *ptr_x++ =
> s->gamma_table[bytestream_get_le16(&src)];
>                          }
> --
> 2.27.0
>
>
ping
Paul B Mahol Nov. 17, 2020, 9:46 a.m. UTC | #2 
LGTM

On Mon, Nov 9, 2020 at 4:37 AM <mindmark@gmail.com> wrote:

> From: Mark Reid <mindmark@gmail.com>
>
> ---
>  libavcodec/exr.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libavcodec/exr.c b/libavcodec/exr.c
> index cf7824402a..e907c5c464 100644
> --- a/libavcodec/exr.c
> +++ b/libavcodec/exr.c
> @@ -1203,7 +1203,7 @@ static int decode_block(AVCodecContext *avctx, void
> *tdata,
>                      }
>                  } else if (s->pixel_type == EXR_HALF) {
>                      // 16-bit
> -                    if (c < 3) {
> +                    if (c < 3 || !trc_func) {
>                          for (x = 0; x < xsize; x++) {
>                              *ptr_x++ =
> s->gamma_table[bytestream_get_le16(&src)];
>                          }
> --
> 2.27.0
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
diff mbox series

Patch

diff --git a/libavcodec/exr.c b/libavcodec/exr.c
index cf7824402a..e907c5c464 100644
--- a/libavcodec/exr.c
+++ b/libavcodec/exr.c
@@ -1203,7 +1203,7 @@  static int decode_block(AVCodecContext *avctx, void *tdata,
                     }
                 } else if (s->pixel_type == EXR_HALF) {
                     // 16-bit
-                    if (c < 3) {
+                    if (c < 3 || !trc_func) {
                         for (x = 0; x < xsize; x++) {
                             *ptr_x++ = s->gamma_table[bytestream_get_le16(&src)];
                         }