[poppler] Implement overprint in qt interface?

[Albert Astals Cid]

El Dilluns, 15 d'octubre de 2012, a les 07:15:51, Thomas Freitag va escriure:
> On 14.10.2012 22:58, Albert Astals Cid wrote:
> > El Diumenge, 14 d'octubre de 2012, a les 19:41:26, Thomas Freitag va 
escriure:
> >> On 14.10.2012 17:21, Albert Astals Cid wrote:
> >>> El Diumenge, 14 d'octubre de 2012, a les 13:47:29, Thomas Freitag va
> > 
> > escriure:
> >>>> Hi folks!
> >>>> 
> >>>> Is there anybody in the community who wants the possibility to simulate
> >>>> overprint in qt library?
> >>>> With the implementation of DeviceN support in splash this is quite easy
> >>>> now, so I can upload a patch.
> >>> 
> >>> Sure, why not? Let's see the patch :-)
> >> 
> >> Okay, here it is.
> > 
> > The two new methods are missing @since markers (and also i think the
> > documentation of the two methods could be a bit more explanatory)
> 
> @since: I thought that You insert it when You commit it. If I would
> insert 0.22.0 You have to change it if You will not have the time to
> commit it :-) or I need to change it and have to upload a new patch if
> You will not have the time :-(

Well, if you do it, it's less work i have to do and thus it's easier i'll have 
time ;-)

> Explanatory: Okay, but this will be hard. For those who know what
> overprint is the documentation is self-explanatory, for others I need to
> write reams. What's about to insert the link to
> http://en.wikipedia.org/wiki/Overprinting?

To be honest doesn't seem to make things much clearer to me :D
Ok, let's leave the docu as it is

> 
> > Also why are you calling it "overprint preview"? How is it a preview?
> 
> To be honest, I haven't really thought about that. Probably I choose
> that name because it is also introduced in GlobalParams with that name.
> But also in the acrobat reader preferences it is called "Use overprint
> preview". And because not all RIPs support it (i.e. RGB printers) it is
> indeed something like a preview (for RIPs which support it).

Ok, let's leave it with that name (but maybe make it a RenderingHint as Adam 
suggests?).

Cheers,
  Albert

> 
> > Given that overprint only works if defined(SPLASH_CMYK) you should make
> > the
> > setter return a boolean that says if the set worked or not (i.e. make it
> > fail if !defined(SPLASH_CMYK))
> 
> Gotcha. I thought about that and disable / enable the option in okular
> if the format generator doesn't support it. But because I'm not familiar
> enough with okular I defered it and then forgot it...
> I'll insert it in QT when I get answers to the other points.
> 
> Cheers,
> Thomas
> 
> > Cheers,
> > 
> >    Albert
> >> 
> >> In a few places it has already a similar
> >> implementation than in bug 50992 (thread safe), because changing the
> >> overprint option also meant that we need a different SplashOutputDev
> >> instance. But You probably also want to test it in okular? Even if we
> >> are here not on an okular list, I attach the okular patch here, too.
> >> Also here, it has already the changes from bug 50992. But because
> >> POPPLER_QT_THREADSAFE is not defined here, it doesn't make any
> >> difference, so I let these changes as they are. BTW, I'm not really
> >> familiar with qt programming, so most changes in okular are more or less
> >> a (hopefully) best guess.
> >> 
> >> Cheers,
> >> Thomas
> >> 
> >>> Cheers,
> >>> 
> >>>     Albert
> >>>> 
> >>>> For everybody who doesn't know anything about overprint I attach three
> >>>> screenshots which shows the implementation in okular.
> >>>> (This is not fake, I made a small apprentice piece today morning :-) )
> >>>> 
> >>>> Cheers,
> >>>> Thomas
> >>> 
> >>> _______________________________________________
> >>> poppler mailing list
> >>> poppler at lists.freedesktop.org
> >>> http://lists.freedesktop.org/mailman/listinfo/poppler
> >>> 
> >>> .
> > 
> > _______________________________________________
> > poppler mailing list
> > poppler at lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/poppler
> > 
> > .
> 
> _______________________________________________
> poppler mailing list
> poppler at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/poppler