diff mbox series

[FFmpeg-devel] avutil/imgutils: fix half-float representation of 0.0625

Message ID 20231228212147.14070-1-cus@passwd.hu
State Accepted
Commit d1785395323f5506ae18ecc4c87d981b9f91af74
Headers show
Series [FFmpeg-devel] avutil/imgutils: fix half-float representation of 0.0625 | expand

Checks

Context Check Description
andriy/make_x86 success Make finished
andriy/make_fate_x86 success Make fate finished
yinshiyou/make_loongarch64 success Make finished
yinshiyou/make_fate_loongarch64 success Make fate finished

Commit Message

Marton Balint Dec. 28, 2023, 9:21 p.m. UTC
We don't have a 16bps float Y format, so no fate changes.

Signed-off-by: Marton Balint <cus@passwd.hu>
---
 libavutil/imgutils.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Marton Balint Jan. 4, 2024, 8:39 p.m. UTC | #1
On Thu, 28 Dec 2023, Marton Balint wrote:

> We don't have a 16bps float Y format, so no fate changes.

Will apply.

Regards,
Marton

>
> Signed-off-by: Marton Balint <cus@passwd.hu>
> ---
> libavutil/imgutils.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libavutil/imgutils.c b/libavutil/imgutils.c
> index ff9ff07961..406e7868b2 100644
> --- a/libavutil/imgutils.c
> +++ b/libavutil/imgutils.c
> @@ -697,7 +697,7 @@ int av_image_fill_black(uint8_t * const dst_data[4], const ptrdiff_t dst_linesiz
>             if (comp.depth < 8 || (fltp && comp.depth != 16 && comp.depth != 32))
>                 return AVERROR(EINVAL);
>             if (fltp)
> -                color = (comp.depth == 16 ? 0x3000 : 0x3D800000); // 0.0625
> +                color = (comp.depth == 16 ? 0x2C00 : 0x3D800000); // 0.0625
>             else
>                 color = 16 << (comp.depth - 8);
>         } else if ((c == 1 || c == 2) && !rgb && !xyz) {
> -- 
> 2.35.3
>
> _______________________________________________
> 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/libavutil/imgutils.c b/libavutil/imgutils.c
index ff9ff07961..406e7868b2 100644
--- a/libavutil/imgutils.c
+++ b/libavutil/imgutils.c
@@ -697,7 +697,7 @@  int av_image_fill_black(uint8_t * const dst_data[4], const ptrdiff_t dst_linesiz
             if (comp.depth < 8 || (fltp && comp.depth != 16 && comp.depth != 32))
                 return AVERROR(EINVAL);
             if (fltp)
-                color = (comp.depth == 16 ? 0x3000 : 0x3D800000); // 0.0625
+                color = (comp.depth == 16 ? 0x2C00 : 0x3D800000); // 0.0625
             else
                 color = 16 << (comp.depth - 8);
         } else if ((c == 1 || c == 2) && !rgb && !xyz) {