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

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

[Marton Balint]

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(-)

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".

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

[Marton Balint]

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".
>
_______________________________________________
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".