GitLab

This partially reverts commit 853e9499, which made CairoOutputDev store
the antialias setting, and applying it to its cairo context.

Instead, just copy the antialias (and font antialias) setting from the
passed cairo context to newly created contexts.  That way, the application
can control the antialias settings.

Use this in pdftocairo to implement its -antialias option.

https://bugs.freedesktop.org/show_bug.cgi?id=94977
poppler/poppler!89

Now you can write globalParams->getUnicodeMap("UTF-8")
which makes much more sense

It comes from a double so it could potentially be negative

Don't make Ascii7Map be static, otherwise it will cache the unicode map
between globlParams and end up with a bad pointer

If we find the constant searching of the unicode in globalparams is a
speed problem we need to apply a similar solution to the one we did for
UTF8

And make the cache just be "infinite", it's not like we support
that many maps or that there's so many used in a given session,
and if they are, well it's good we cached them

All the unicode maps we support use about 2MB of memory, but PSOutputDev
is the only one that loads "random" unicodeMaps so to load them all
you'd had to print lots of different documents with fonts with lots of
different font encodings, so it seems like a not very likely situation
and the code gets simplified a bit

Move the initer of the cpp frontend into the core to able to reuse it for the qt5 and glib frontends.

Remove LinkAction::setNextActions as this setter should not be used outside of the ::parseAction method.

The convertLinkActionToLink method deep-copies every poppler LinkAction
object into a Link object as used by the Qt5 frontend.  With one
exception: LinkOCGState objects are not deep-copied, but pointers to
them are stored instead.  This makes for a memory management that is
inconsistent, difficult to understand, and prone to memory errors.
The present patch introduces deep-copying for LinkOCGState objects
as well.

... rather than a pointer.  This makes it clearer that the
result is borrowed, and should not be shallow-copied.

Rather than a pointer to a std::vector.  This removes one
layer of indirection when accessing the vector.

Makes the code a bit simpler

GlobalParams::getUnicodeMap was just calling
GlobalParams::getUnicodeMap2 so move the code of this one to the other
one and save ourselves having one function that just calls the other

To make it clear that it is an owning pointer.

oss-fuzz/19630