Hi Andreas,
On Tue, Jul 2, 2024, 11:48 Andreas Rheinhardt <
andreas.rheinha...@outlook.com> wrote:
> Sean McGovern:
> > Hi Leo,
> >
> >
> > On Tue, Jul 2, 2024, 09:44 Leo Izen <leo.i...@gmail.com> wrote:
> >
> >> When mDCv support was added, there was a typo in both variable names
> >> and also the MKTAG itself, incorrectly listing it as mDVc. The tag name
> >> stands for Mastering Display Color Volume so mDCv is correct.
> >>
> >> Typo originally introduced in 78949041417caaef0c82b2b23d7defdd88aa2378.
> >>
> >> Signed-off-by: Leo Izen <leo.i...@gmail.com>
> >> Reported-by: Ramiro Polla <ramiro.po...@gmail.com>
> >> ---
> >> libavcodec/pngenc.c | 18 +++++++++---------
> >> 1 file changed, 9 insertions(+), 9 deletions(-)
> >>
> >> diff --git a/libavcodec/pngenc.c b/libavcodec/pngenc.c
> >> index 89c3d5b83e..7ae16fa2c5 100644
> >> --- a/libavcodec/pngenc.c
> >> +++ b/libavcodec/pngenc.c
> >> @@ -450,17 +450,17 @@ static int encode_headers(AVCodecContext *avctx,
> >> const AVFrame *pict)
> >>
> >> side_data = av_frame_get_side_data(pict,
> >> AV_FRAME_DATA_MASTERING_DISPLAY_METADATA);
> >> if (side_data) {
> >> - AVMasteringDisplayMetadata *mdvc = (AVMasteringDisplayMetadata
> *)
> >> side_data->data;
> >> - if (mdvc->has_luminance && mdvc->has_primaries) {
> >> + AVMasteringDisplayMetadata *mdcv = (AVMasteringDisplayMetadata
> *)
> >> side_data->data;
> >> + if (mdcv->has_luminance && mdcv->has_primaries) {
> >> for (int i = 0; i < 3; i++) {
> >> - AV_WB16(s->buf + 2*i,
> >> PNG_Q2D(mdvc->display_primaries[i][0], 50000));
> >> - AV_WB16(s->buf + 2*i + 2,
> >> PNG_Q2D(mdvc->display_primaries[i][1], 50000));
> >> + AV_WB16(s->buf + 2*i,
> >> PNG_Q2D(mdcv->display_primaries[i][0], 50000));
> >> + AV_WB16(s->buf + 2*i + 2,
> >> PNG_Q2D(mdcv->display_primaries[i][1], 50000));
> >> }
> >> - AV_WB16(s->buf + 12, PNG_Q2D(mdvc->white_point[0], 50000));
> >> - AV_WB16(s->buf + 14, PNG_Q2D(mdvc->white_point[1], 50000));
> >> - AV_WB32(s->buf + 16, PNG_Q2D(mdvc->max_luminance, 10000));
> >> - AV_WB32(s->buf + 20, PNG_Q2D(mdvc->min_luminance, 10000));
> >> - png_write_chunk(&s->bytestream, MKTAG('m', 'D', 'V', 'c'),
> >> s->buf, 24);
> >> + AV_WB16(s->buf + 12, PNG_Q2D(mdcv->white_point[0], 50000));
> >> + AV_WB16(s->buf + 14, PNG_Q2D(mdcv->white_point[1], 50000));
> >> + AV_WB32(s->buf + 16, PNG_Q2D(mdcv->max_luminance, 10000));
> >> + AV_WB32(s->buf + 20, PNG_Q2D(mdcv->min_luminance, 10000));
> >> + png_write_chunk(&s->bytestream, MKTAG('m', 'D', 'C', 'v'),
> >> s->buf, 24);
> >> }
> >> }
> >>
> >> --
> >> 2.45.2
> >>
> >> _______________________________________________
> >> ffmpeg-devel mailing list
> >> ffmpeg-devel@ffmpeg.org
> >> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> >>
> >> To unsubscribe, visit link above, or email
> >> ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
> >>
> >
> > This did also land on the 7.0 branch so I assume it will get a backport
> if
> > approved?
> >
>
> Why should it be backported? This patch has no user-facing consequences.
>
> - Andreas
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
>
Even though it fixes the written tag in the call to png_write_chunk()?
-- Sean McGovern
>
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
