[FFmpeg-devel] [PATCH 2/2] libavcodec/mjpeg: remove YUVJ mentions

Vittorio Giovara vittorio.giovara at gmail.com
Fri Dec 8 17:11:19 EET 2017

> On 12/8/17, Paul B Mahole *<onemda at gmail.com <http://ffmpeg.org/mailman/listinfo/ffmpeg-devel>>* wrote:
>*> This will basically break everyone encoding mjpeg right now, since it
*>*> suddenly only accepts different formats without any common-ground
*>*> before/after.
*>*> Furthermore, there is no replacement for the indication that this
*>*> encoder wants full-range data, which the old pixfmts indicated.
> So I will add .color_range to AVCodec
> 0 means encoder supports both.
> Is that ok?

I tried in the past and it is indeed possible, however it doesn't really
work: going that route you'll end up adding all the color properties,
including chroma location.
As much as I despise the J formats, it's a hacky solution at best leading
to a very confusing API, making the negotiation and conversion unnecessary
If we were to break this feature, I'd suggest going the full route of
adding a PixelFormaton and work on a sws alternative (one is allowed to
dream no?).

More information about the ffmpeg-devel mailing list