Commit Graph

66 Commits

Author SHA1 Message Date
Jeff Muizelaar
99eabd8851 Bug 764181. Keep the output of the TRC between 0 and 1. r=bgirard
Section 10.16: The domain and range of each function shall be [0,0 1,0].
Any function value outside the range shall be clipped to the range of the function.

--HG--
extra : rebase_source : 165927afb271980259c021fa0af25787be4a7ad3
2012-06-07 16:47:34 -04:00
Gervase Markham
d4eb7d5782 Bug 759095 - upgrade license to MPL 2, and other licensing cleanups.
--HG--
extra : rebase_source : da55a4937383eda2baf7c9a362501da8ee664146
2012-05-29 16:52:43 +01:00
Benoit Girard
7fdc677b78 Bug 752254 - qcms - Prefer precache output curves even if a A2B/B2A CLUT is present. r=jmuizelaar
--HG--
extra : rebase_source : fb5135434d7c7c80656a54e21b6e3b6216210b71
2012-05-22 14:51:29 -04:00
Mike Hommey
40b66bc763 Bug 755277 - Avoid duplicate rules after bug 748001. r=ted 2012-05-15 19:01:01 +02:00
Makoto Kato
e6f6d1c3f1 Bug 719383 - Remove DIST_INSTALL from gfx/qcms/Makefile. r=jmuizelaar 2012-02-13 14:00:06 +09:00
Bill McCloskey
af6c302c9e Bug 705332 - Re-enable printing filename in quiet builds (r=njn) 2011-12-08 10:14:06 -08:00
aceman
03b5fc2b81 Bug 704210 - Add license headers to files gfx/gl/Makefile.in and gfx/thebes/Makefile.in. r=joe 2011-12-03 13:04:52 +01:00
Nicholas Nethercote
a4c1789959 Bug 703456 - Kill REPORT_BUILD; kill it dead. r=ted. 2011-11-21 18:47:54 -08:00
Benoit Girard
0cda6c0e73 Bug 701348 - qcms_transform_precacheLUT_float did not properly free dest buffer. r=jmuizelaar 2011-11-10 08:28:58 -05:00
Gregory Szorc
dd260c7514 Merge b-s to m-c 2011-11-10 15:15:29 -08:00
Gregory Szorc
f9737cefa1 Bug 698248 - Remove unused FORCE_USE_PIC variable; r=khuey 2011-11-01 15:45:23 -07:00
Jeff Muizelaar
b4a44ef015 Bug 700773. qcms: cleanup up some integer types. r=bgirard
This fixes a couple of warnings in clang and makes things more consistent.

--HG--
extra : rebase_source : feac84090f270461a73de955902bc016b5835616
2011-11-08 15:43:33 -05:00
Yati Sagade
83f095410d Bug 550969 - Fix white_point_from_temp to handle out of range input. r=jmuizelaar 2011-10-14 14:58:34 -04:00
Landry Breuil
4c9bbf5e1f Bug 651444 - inttypes.h is needed on OpenBSD to get uintptr_t; r=jmuizelaar 2011-09-27 19:59:34 +01:00
Landry Breuil
00c4f00daf Bug 651444 - Dont redefine intX_t types on OpenBSD; r=jmuizelaar 2011-09-20 17:32:18 +01:00
Jeff Muizelaar
7cb83bc84a Bug 676747. qcms: Fix a warning about using the wrong function type. r=bgirard
--HG--
extra : rebase_source : 177b781ea8f8cc350283bd082cbd3600433c3f82
2011-08-05 09:49:45 -04:00
Benoit Girard
3cde961989 Bug 674230 - Merge from qcms github branch v4; r=jmuizelaar 2011-07-27 10:11:19 -04:00
Jeff Muizelaar
1a508abb82 Bug 672280. qcms: Use C99-style flexible arrays. r=bgirard
This is in preparation for adding more of these for the
upcoming ICCv4 parsing.
2011-07-19 11:17:47 -04:00
Jeff Muizelaar
2dfcf68c91 qcms: avoid unnecessary promotion to double. r=ehsan 2011-05-10 12:01:48 -04:00
Jeff Muizelaar
cd58cf15e6 Bug 605179. qcms: revert build warning fixes. r=ehsan
0ec22e77aefc fixed some build warnings that no longer happen because pedantic is disabled.
2011-05-10 11:55:39 -04:00
Jeff Muizelaar
4b4fccbd45 Bug 648446. Remove -pedantic from CFLAGS in qcms. r=khuey
-pedantic causes warnings that we don't care about
2011-04-08 11:34:16 -07:00
Jeff Muizelaar
c64cd62159 Bug 646321. qcms: Avoid implicit conversion to double. r=bjacob
The double literal will cause us to unnecessarily convert to double.
2011-04-07 16:59:12 -07:00
Makoto Kato
14ea2bfa7f Bug 613440. Make loading ICM profiles from non-ASCII paths work. r=jmuizelaar 2011-03-30 13:40:47 -04:00
Jeff Muizelaar
20900cc870 Bug 506207 - Check the return value of fread to make sure the readed data is valid; r,a=joe 2011-02-28 13:01:05 -08:00
Ms2ger
3ec44ade0c Bug 605179 - Fix build warnings in gfx/; r=roc a=bsmedberg
--HG--
extra : rebase_source : f9bd24d5f4bc6124b431ce22e6ac72bd218dc208
2010-12-05 18:38:12 +00:00
Jacek Caban
f85ade1e8b Bug 594769 - qcms fails to compile on mingw r=jmuizelaar a=joe 2010-10-14 21:00:22 +00:00
Benoit Girard
fa72189e5f Bug 575142 - qcms_transform_create should use transform_free. r=jmuizellar 2010-07-24 20:10:15 -04:00
Daniel Holbert
f535a780b9 Bug 503279: disable spammy 'missing initializer' warning in imported code within gfx. r=jrmuizel 2010-07-13 11:50:18 -07:00
Justin Wood
ce097ec76a Bug 467051 - Introduce GLOBAL_DEPS to build system (gfx/cairo) r=vlad, feedback+=ted
--HG--
extra : rebase_source : 3dee9c67387cb9b72dd81da6d52ad996afa2bfae
2010-05-30 22:49:32 -04:00
Jacek Caban
e8ee0dbbf6 Bug 569475 - Fix qcms linking failure on mingw-w64. r=jmuizelaar
--HG--
extra : rebase_source : 07a082564d002faeaedc4fb57911ec9d64b605fd
2010-06-07 10:16:22 +02:00
Jeff Muizelaar
e88bb567f0 Bug 548612. qcms: Reduce size of output lookup tables. r=bgirard
We currently use a larger output lookup table than we probably need. Switch to
a common define for the table size and lower it. The should also give a small
improvement to startup time because we have fewer lookup table entries to
compute.
2010-06-04 14:48:30 -04:00
Daniel Holbert
463f0a75f5 Bug 561787: Add missing #include to iccread.c, to fix compile warnings about declarations of memcpy. r=roc 2010-04-28 23:13:13 -07:00
Michael Wu
298d78d642 Bug 556405 - Support building qcms on Android, r=jrmuizel 2010-04-05 17:38:43 -07:00
Jeff Muizelaar
c887d8596e Bug 550188. qcms: remove an empty loop that annoyed coverity. r=bgirard 2010-03-04 12:13:13 -05:00
Steve Snyder
929a12a259 qcms: Replace second green table release with release of blue table
Cut 'n' paste gone awry: green output table freed instead of blue.

That is, there exists a release for red, 2 releases for green, and none for
blue.
2010-02-18 15:21:11 -05:00
Makoto Kato
3c3953210e Bug 536725 - Disable SSE1 code on Windows x64 r=jmuizelaar 2010-02-01 17:43:21 +09:00
Shailen
bfe1e71cd9 Bug 523848 - qcms typedef clashes on AIX with <sys/types.h>. r=jmuizelaar, sr=roc 2010-01-20 11:21:19 +01:00
Ginn Chen
a71851dfb1 Bug 537070 qcms failed to build with -m64 on Solaris x86_64 r=jrmuizel 2010-01-06 15:02:04 +08:00
Ginn Chen
8042e8805d Bug 522440 QCMS: Make sse/sse2 code built with Sun Studio 12 on Solaris r=jmuizelaar, ted.mielczarek 2009-11-20 14:15:39 +08:00
Jeff Muizelaar
26b24b8c8d Backout 8a790e81279e (bug 528322) because wince doesn't have powf.
The link error was obscured by other errors when I ran this on try...
2009-11-13 11:53:12 -05:00
Jeff Muizelaar
9c3353941f Bug 528322. qcms: Avoid unneeded precision during table creation. r=joe
Switch from pow() to powf() because it's faster and we don't need the
additional precision.  Also avoid unnecessary conversion to and from doubles by
using float constants instead of doubles.
2009-11-13 10:14:13 -05:00
Jeff Muizelaar
4080919147 Bug 488800. qcms: accept ICCv4 profiles. r=bobbyholley
ICCv4 profiles are a superset of ICCv2 so it shouldn't hurt us to accept them and
just treat them the same way we do ICCv2 profiles.
2009-10-02 17:57:25 -04:00
Steve Snyder
6a4d539353 Bug 512865. qcms: Improve SSE2 performance, add SSE support. r=jrmuizel
This patch greatly improves the performance of QCMS transformations on x86 &
x86_64 systems.  Some notes:

0. On 32-bit x86 systems it does runtime selection between non-SIMD, SSE, and
SSE2 code paths.

1. On x86_64 systems the SSE2 code path is always taken.  The non-SIMD and SSE
code paths are left intact, but contemporary versions of the GCC and MSVC
compilers will see that they cannot be reached and optimize them away.

2. The execution of the SSE2 code path is reduced by 67%, relative to the
original Intel/Microsoft formatted ASM code.  The relative performance is seen
on a Pentium4 (Northwood) 2.4GHz CPU with DDR1 RAM.

3. The SSE code path provides a 80% reduction in execution time, relative to
the non-SIMD code path.  The relative performance is seen on a Pentium3
(Coppermine) 1.26GHz CPU with SDRAM.

4. The code has been split out into separate files so that it can be built
with different cflags (-msse, and -msse2) when using gcc.

5. Try to land again, this time with __attribute__((__force_align_arg_pointer__))
to avoid crashes on linux.
2009-10-01 14:19:41 -04:00
Jeff Muizelaar
fe18ffef29 Backed out changeset cb4f078cc8cb (bug 512865)
Was causing crashes on the leak test box.
2009-09-24 23:35:53 -04:00
Steve Snyder
0ed3ad35a2 Bug 512865. qcms: Improve SSE2 performance, add SSE support. r=jrmuizel
This patch greatly improves the performance of QCMS transformations on x86 &
x86_64 systems.  Some notes:

0. On 32-bit x86 systems it does runtime selection between non-SIMD, SSE, and
SSE2 code paths.

1. On x86_64 systems the SSE2 code path is always taken.  The non-SIMD and SSE
code paths are left intact, but contemporary versions of the GCC and MSVC
compilers will see that they cannot be reached and optimize them away.

2. The execution of the SSE2 code path is reduced by 67%, relative to the
original Intel/Microsoft formatted ASM code.  The relative performance is seen
on a Pentium4 (Northwood) 2.4GHz CPU with DDR1 RAM.

3. The SSE code path provides a 80% reduction in execution time, relative to
the non-SIMD code path.  The relative performance is seen on a Pentium3
(Coppermine) 1.26GHz CPU with SDRAM.

4. The code has been split out into separate files so that it can be built
with different cflags (-msse, and -msse2) when using gcc.
2009-09-24 11:58:24 -04:00
Vladimir Vukicevic
56304c7aa1 Bug 504766. qcms: Fix unaligned reads in qcms. r=jrmuizel
Offsets are not guaranteed to be aligned, so don't assume that they are.
memcpy instead of just casting and dereferencing.
2009-09-08 11:36:28 -04:00
timeless@mozdev.org
86ae0b7a12 Bug 491195 fix warnings in qcms
mostly comparisons between unsigned and signed
also an incomplete initializer
and a warning about using C++ style variables
r=jrmuizel
2009-07-22 17:31:03 -05:00
Jeff Muizelaar
f1000364be Bug 504888. qcms: Avoid integer overflow when checking buffer bounds. r=bobbyholley
Found by Chris Evans
2009-07-21 23:32:27 -04:00
Jeff Muizelaar
329795d916 Bug 498245. Check that the profile is an RGB profile before checking if it's bogus. r=bholley
This avoids unitialized reads on non-RGB profiles.
2009-07-02 11:34:08 -04:00
Jeff Muizelaar
a3a14052d6 Bug 498245. Profiles with negative colorant tristiumlus values are bogus. r=bholley
Make sure we reject these profiles.
2009-07-02 11:34:06 -04:00