[FFmpeg-devel] [PATCH] use JPEG color range for RoQ

[Eric Lasota]

Additional patch to set color_range on decode.


On Sun, Jul 6, 2014 at 5:41 PM, Michael Niedermayer <michaelni at gmx.at>
wrote:

> On Sun, Jul 06, 2014 at 04:17:16PM -0400, Eric Lasota wrote:
> > The first-party codec uses JPEG YUV, see values in ROQ_GenYUVTables:
> >
> https://github.com/id-Software/Quake-III-Arena/blob/master/code/client/cl_cin.c
> >
> > Test with idlogo.roq:
> > Reference implementation: http://i.imgur.com/QeywyIr.png (enlarged to
> work
> > around a rendering bug)
> > ffmpeg without patch: http://i.imgur.com/Q7ZHBPd.png
> > ffmpeg with patch: http://i.imgur.com/ZJE01uy.png
>
> >  roqvideodec.c |    2 +-
> >  roqvideoenc.c |    2 +-
> >  2 files changed, 2 insertions(+), 2 deletions(-)
> > a4f768fff83e72ea7f356c2b12b0ed6170d043bc
>  0001-use-JPEG-color-range-for-RoQ.patch
> > From ca0e542adf924b42a8502e313fa911576924978f Mon Sep 17 00:00:00 2001
> > From: ejlasota <ejlasota at gmail.com>
> > Date: Sun, 6 Jul 2014 15:42:43 -0400
> > Subject: [PATCH] use JPEG color range for RoQ
>
> applied
>
> thanks
>
> [...]
>
> --
> Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
>
> Rewriting code that is poorly written but fully understood is good.
> Rewriting code that one doesnt understand is a sign that one is less smart
> then the original author, trying to rewrite it will not make it better.
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-set-JPEG-output-color-range-on-RoQ.patch
Type: application/octet-stream
Size: 688 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140706/66569c38/attachment.obj>