<div dir="auto">Sorry, i wanted to send it to the list, too</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">---------- Forwarded message ---------<br>Von:<strong class="gmail_sendername" dir="auto">Thomas Freitag</strong> <span dir="ltr"><<a href="mailto:thomas.freitag.bbr@gmail.com">thomas.freitag.bbr@gmail.com</a>></span><br>Date: Do., 25. Apr. 2019, 07:42<br>Subject: Re: [poppler] splashModeDeviceN8 in two switch<br>To: Albert Astals Cid <<a href="mailto:aacid@kde.org">aacid@kde.org</a>><br></div><br><br><div dir="auto">Your argument is wrong, Albert.<div dir="auto">Even if we remove the compiler directive, "noone" will use this mode by default, because the default rendering is RGB. So you don't need more memory for rendering and it doesn't become slower. Just the code need more memory.</div><div dir="auto">But the "professionell" user would have the ability to simulate overprint. As far as i remember i implemented an overprint option in pdftoppm for this purpose. You can see the difference if you run the ghent test suite with and without this switch.</div><div dir="auto"><br></div><div dir="auto">Cheers,</div><div dir="auto">Thomas</div><div dir="auto"><br></div><div dir="auto">PS: Not only William uses it, i do it too</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Albert Astals Cid <<a href="mailto:aacid@kde.org" target="_blank" rel="noreferrer">aacid@kde.org</a>> schrieb am Mi., 24. Apr. 2019, 19:40:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">El dimarts, 23 d’abril de 2019, a les 18:11:04 CEST, Adam Reichold va escriure:<br>
> Hello,<br>
> <br>
> thinking about this again with a bit of distance, should we maybe remove<br>
> the `SPLASH_CMYK` preprocessor flag entirely? I am assuming the reasons<br>
> for having this upstream is to keep the code from bit rotting as<br>
> Poppler's internals change? If nobody builds this, then this will not<br>
> happen. (CI checks improve things, but it will still be "late" feedback<br>
> instead of being part of the initial edit-compile-test cycle. Also, this<br>
> will not reach e.g. clusterfuzz which thereby will not verify that code.)<br>
> <br>
> Are there any downsides to removing the conditional compilation? Does<br>
> this break code that does not request the CMYK handling explicitly? If<br>
> so, could we fix this via the internal API instead of hard-coded it at<br>
> compile time?<br>
<br>
The main problem i see is that it makes SplashColor for from 4 bytes to 8 bytes, so may make things slower/use more memory for a feature "noone uses".<br>
<br>
Cheers,<br>
  Albert<br>
<br>
P.S: Of course the "noone uses" is a bit of a self fulfilling prophecy if we don't enable it<br>
<br>
> <br>
> Best regards,<br>
> Adam<br>
> <br>
> Am 22.04.19 um 13:14 schrieb Albert Astals Cid:<br>
> > This is mostly for William since AFAIK he's the "only" one using SPLASH_CMYK.<br>
> > <br>
> > I've tried to enable SPLASH_CMYK on the CI and it's loudly complaining that the splashModeDeviceN8 cases are missing in SplashOutputDev.cc<br>
> > <br>
> > <a href="https://gitlab.freedesktop.org/aacid/poppler/-/jobs/253647" rel="noreferrer noreferrer noreferrer" target="_blank">https://gitlab.freedesktop.org/aacid/poppler/-/jobs/253647</a><br>
> > <br>
> > Any suggestion of what the code should be?<br>
> > <br>
> > Should they just be the same as case splashModeCMYK8: ? The first one looks like it may make sense, but not so sure about the second.<br>
> > <br>
> > Cheers,<br>
> >   Albert<br>
> > <br>
> > <br>
> > _______________________________________________<br>
> > poppler mailing list<br>
> > <a href="mailto:poppler@lists.freedesktop.org" rel="noreferrer noreferrer" target="_blank">poppler@lists.freedesktop.org</a><br>
> > <a href="https://lists.freedesktop.org/mailman/listinfo/poppler" rel="noreferrer noreferrer noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/poppler</a><br>
> > <br>
> <br>
> <br>
<br>
<br>
<br>
<br>
_______________________________________________<br>
poppler mailing list<br>
<a href="mailto:poppler@lists.freedesktop.org" rel="noreferrer noreferrer" target="_blank">poppler@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/poppler" rel="noreferrer noreferrer noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/poppler</a></blockquote></div>
</div>