[Poppler-bugs] [Bug 50992] library should be thread-safe

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=50992

--- Comment #38 from Thomas Freitag <Thomas.Freitag at alfa.de> 2012-08-23 08:57:32 UTC ---
How should we continue now (and should we continue or leave it as a study
only)?

1. The just uploaded patch is complete and regtested for page rendering with
splash. What is with other devices? I could implement it for cairo (if the
cairo library is thread safe itself and does i.e. not use static variables) and
test if with a testcase for pdftocairo, but I'm not sure about other devices,
and i.e. in my opinion it makes no sense to have a thread safe page rendering
in PSOutputDev.

2. We still have the open question if we copy the xref all the time and do not
use doc->getXRef() anymore. I think it's not too expensive to copy it all the
time and would implement it if all agree. I hadn't done that until now because
it was easier to compare differences.

3. I think that a PDFDoc instance is what should be able to share in different
threads, so I offer to have a additional look into the PDFDoc implementation
which methods are changing private members and lock these methods, but I have
no idea how to test such changes.

Any comments are welcome, even if the main opinion would be to leave this as a
study. It was more work than I thought, but until now I did it because of
theoretical interests.

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.