<html> <head> <base href="https://bugs.freedesktop.org/" /> </head> <body> <div> <a class="bz_bug_link bz_status_NEW " title="NEW --- - 100% CPU usage" href="https://bugs.freedesktop.org/show_bug.cgi?id=66928#c23">Comment # 23</a> on <a class="bz_bug_link bz_status_NEW " title="NEW --- - 100% CPU usage" href="https://bugs.freedesktop.org/show_bug.cgi?id=66928">bug 66928</a> from <a class="email" href="mailto:Thomas.Freitag@alfa.de" title="Thomas Freitag <Thomas.Freitag@alfa.de>"> Thomas Freitag</a> <pre>Created <a href="attachment.cgi?id=84195" name="attach_84195" title="use getRGBLine, getCMYKLine and getDeviceNLine in case of images if they ara available">attachment 84195</a> <a href="attachment.cgi?id=84195&action=edit" title="use getRGBLine, getCMYKLine and getDeviceNLine in case of images if they ara available">[details]</a> <a href='page.cgi?id=splinter.html&bug=66928&attachment=84195'>[review]</a> use getRGBLine, getCMYKLine and getDeviceNLine in case of images if they ara available I finished my qork quicker than expected, so I had some time to look into it already, and here is the patch. Yes, Albert, You were true (as always): the usage of getRGBLine speed up the rendering in case of images. I think that at least lcms2 does than some caching internally. But to complete it also in the CMYK and DeviceN area, we need for them the correcponding functions, too. So this patch does not only use getRGBLine in SplashOutputDev, but it also implements the other needed functions. And even if I still think we need the cache i.e. for the getRGB() function, I've removed it from this patch, because we don't need it to solve this bug now. On the other hand I don't mark it as obsolete, it's up to You, Albert, to think once again about it.</pre> </div> <hr> You are receiving this mail because: <ul> <li>You are the assignee for the bug.</li> </ul> </body> </html>