[FFmpeg-devel] [PATCH 10/24] avcodec/mjpegdec: replace YUVJ pixel formats

[Michael Niedermayer]

On Wed, Dec 13, 2017 at 04:44:26PM -0300, James Almer wrote:
> On 12/13/2017 4:22 PM, Michael Niedermayer wrote:
> > On Wed, Dec 13, 2017 at 11:59:26AM +0100, Paul B Mahol wrote:
> >> Signed-off-by: Paul B Mahol <onemda at gmail.com>
> >> ---
> >>  libavcodec/mjpegdec.c                | 18 +++++++++---------
> >>  libavcodec/tdsc.c                    |  2 +-
> >>  tests/fate/vcodec.mak                |  4 ++--
> >>  tests/ref/fate/api-mjpeg-codec-param |  4 ++--
> >>  tests/ref/fate/exif-image-embedded   |  2 +-
> >>  tests/ref/fate/exif-image-jpg        |  2 +-
> >>  6 files changed, 16 insertions(+), 16 deletions(-)
> > 
> > this breaks ffplay playing a mjpeg in avi
> > 
> > ./ffmpeg -i matrixbench_mpeg2.mpg -vcodec mjpeg -t 0.5  -qscale 1 mjpeg.avi
> > ./ffplay mjpeg.avi
> > 
> > the output of ffplay looks darker than it should be
> 
> Without this patch swscale emits a warning about deprecated pixel
> format. With it, the warning is gone.
> 
> Guess swscale needs to be adapted in some way before the same is done to
> the decoders?

To clarify the report, the mjpeg.avi is the same before and after the
patch, the difference is in the ffplay command

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Why not whip the teacher when the pupil misbehaves? -- Diogenes of Sinope
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20171213/22b64364/attachment.sig>