<html>
<head>
<base href="https://bugs.freedesktop.org/" />
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW --- - Poppler ignores rendering intents and always uses INTENT_RELATIVE_COLORIMETRIC"
href="https://bugs.freedesktop.org/show_bug.cgi?id=34053#c30">Comment # 30</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW --- - Poppler ignores rendering intents and always uses INTENT_RELATIVE_COLORIMETRIC"
href="https://bugs.freedesktop.org/show_bug.cgi?id=34053">bug 34053</a>
from <span class="vcard"><a class="email" href="mailto:Thomas.Freitag@alfa.de" title="Thomas Freitag <Thomas.Freitag@alfa.de>"> <span class="fn">Thomas Freitag</span></a>
</span></b>
<pre>(In reply to <a href="show_bug.cgi?id=34053#c29">comment #29</a>)
<span class="quote">> Hmmm, your patch makes it so that in the first Gfx constructor we first do
>
> : : :
> as part of the constructor, this used to be the other way around. Do we
> need/want this?</span >
Hmmh, I missed that, when I tried to remove redundant code. It's probably
better to go back to the old constructors and then just call something like
initDisplayProfile() at the end of both constructors, where initDisplayProfile
do the new stuff. Is it okay for You that I change my patch after You commit
<a class="bz_bug_link
bz_status_NEW "
title="NEW --- - Cache lcms transformed values for ICC colorspaces in getRGB(), getCMYK() and getGray()"
href="show_bug.cgi?id=68420">bug 68420</a>?</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>