[FFmpeg-devel] [PATCH] Add a gamma flag to exr loader to avoid banding

[Michael Niedermayer]

On Thu, May 08, 2014 at 01:27:56PM -0300, Gonzalo Garramuno wrote:
> On 08/05/14 12:29, Michael Niedermayer wrote:
> >
> >still fails:
> >
> >also theres still a float in there in the gamma=1 path:
> >"if ( t.f <= 0.0f ) {"
> >
> >--- ./tests/ref/fate/exr-slice-raw      2014-05-08 17:24:22.886249292 +0200
> >+++ tests/data/fate/exr-slice-raw       2014-05-08 17:25:54.690251226 +0200
> >@@ -1,2 +1,2 @@
> >  #tb 0: 1/25
> >-0,          0,          0,        1,  3169800, 0x1f656d7a
> >+0,          0,          0,        1,  3169800, 0x6a356d0d
> >Test exr-slice-raw failed. Look at tests/data/fate/exr-slice-raw.err for details.
> >
> >[...]
> I made a distclean rebuild and now I got the old checksums just like
> above.  In summary, with the gamma table as in the attachment, the
> chceksums must not be changed.  This is what I expected when I
> created the gamma=1 path.
> 
> 

seems to work
applied
thx

[...]
-- 
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.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140508/eba01970/attachment.asc>