On 29 December 2012 10:22, Richard Hughes <hughsi...@gmail.com> wrote:
> I've attached a patch which fixes the issue for me.

And another patch attached:

commit 17e2e35829993b7f7cbbf7ffe81859ee2f816f80
Author: Richard Hughes <rich...@hughsie.com>
Date:   Tue Jan 1 17:13:10 2013 +0000

    Fix the Rec. 709 parametric curve type example in the test program

    The code is commented out, but it stops someone trying to do the same as me
    making the same mistake.

I spent a good couple of hours looking at the maths before the mistake
popped out at me. I had to manually invert the formula and plot the
graph in OpenOffice to verify the sign was indeed wrong. It's up to
you if you just want to remove the #ifdef0 code, but having a wrong
example is probably a bad thing :)

I've added a a Rec. 709 profile to shared-color-profiles now. Thanks. :)

Richard.

Attachment: 0003-Fix-the-Rec.-709-parametric-curve-type-example-in-th.patch
Description: Binary data

------------------------------------------------------------------------------
Master SQL Server Development, Administration, T-SQL, SSAS, SSIS, SSRS
and more. Get SQL Server skills now (including 2012) with LearnDevNow -
200+ hours of step-by-step video tutorials by Microsoft MVPs and experts.
SALE $99.99 this month only - learn more at:
http://p.sf.net/sfu/learnmore_122512
_______________________________________________
Lcms-user mailing list
Lcms-user@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/lcms-user

Reply via email to