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

[Vittorio Giovara]

> 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
complex.
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?).
-- 
Vittorio