[FFmpeg-devel] [PATCH] [PATCH] avcodec: Add more kCVImageBufferColorPrimaries to videotoolboxenc

Rick Kern kernrj at gmail.com
Sun Nov 17 23:32:54 EET 2019


On Sun, Nov 17, 2019 at 10:25 AM Nomis101 <Nomis101 at web.de> wrote:

> Am 17.11.19 um 01:05 schrieb Richard Kern:
> >
> >> On Nov 16, 2019, at 7:03 PM, Nomis101 <Nomis101 at web.de> wrote:
> >>
> >> Thanks for reviewing. Would be nice, if somebody could push to master
> then. I can't.
> >>
> >>
> > I’ll test it out tomorrow and push.
>
> Cool, thank you very much. :-)
>
Thanks, pushed.


>
> >
> >>> Am 15.11.19 um 15:58 schrieb Limin Wang:
> >>>> On Sun, Nov 03, 2019 at 01:20:38AM +0100, Nomis101 wrote:
> >>>> ---
> >>>> libavcodec/videotoolboxenc.c | 8 ++++++++
> >>>> 1 file changed, 8 insertions(+)
> >>>>
> >>>> diff --git a/libavcodec/videotoolboxenc.c
> b/libavcodec/videotoolboxenc.c
> >>>> index 40a7f643e0..cc08cf6a50 100644
> >>>> --- a/libavcodec/videotoolboxenc.c
> >>>> +++ b/libavcodec/videotoolboxenc.c
> >>>> @@ -894,6 +894,14 @@ static int get_cv_color_primaries(AVCodecContext
> *avctx,
> >>>>             *primaries = NULL;
> >>>>             break;
> >>>>
> >>>> +        case AVCOL_PRI_BT470BG:
> >>>> +            *primaries = kCVImageBufferColorPrimaries_EBU_3213;
> >>>> +            break;
> >>>> +
> >>>> +        case AVCOL_PRI_SMPTE170M:
> >>>> +            *primaries = kCVImageBufferColorPrimaries_SMPTE_C;
> >>>> +            break;
> >>>> +
> >>> lgtm
> >>>
> >>>>         case AVCOL_PRI_BT709:
> >>>>             *primaries = kCVImageBufferColorPrimaries_ITU_R_709_2;
> >>>>             break;
> >>>> --
> >>>> 2.21.0 (Apple Git-122)
> >>>>
> >>>> _______________________________________________
> >>>> 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".
> >>> _______________________________________________
> >>> 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".
> >> _______________________________________________
> >> 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".
> > _______________________________________________
> > 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".
>
>
> _______________________________________________
> 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".


More information about the ffmpeg-devel mailing list