<html>
<head>
<base href="https://bugs.freedesktop.org/" />
</head>
<body><span class="vcard"><a class="email" href="mailto:Thomas.Freitag@alfa.de" title="Thomas Freitag <Thomas.Freitag@alfa.de>"> <span class="fn">Thomas Freitag</span></a>
</span> changed
<a class="bz_bug_link
bz_status_NEW "
title="NEW --- - make qt4 frontend thread-safe"
href="https://bugs.freedesktop.org/show_bug.cgi?id=59927">bug 59927</a>
<br>
<table border="1" cellspacing="0" cellpadding="8">
<tr>
<th>What</th>
<th>Removed</th>
<th>Added</th>
</tr>
<tr>
<td style="text-align:right;">Attachment #73724 is obsolete</td>
<td>
</td>
<td>1
</td>
</tr></table>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW --- - make qt4 frontend thread-safe"
href="https://bugs.freedesktop.org/show_bug.cgi?id=59927#c12">Comment # 12</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW --- - make qt4 frontend thread-safe"
href="https://bugs.freedesktop.org/show_bug.cgi?id=59927">bug 59927</a>
from <span class="vcard"><a class="email" href="mailto:Thomas.Freitag@alfa.de" title="Thomas Freitag <Thomas.Freitag@alfa.de>"> <span class="fn">Thomas Freitag</span></a>
</span></b>
<pre>Created <span class=""><a href="attachment.cgi?id=73820" name="attach_73820" title="fix font info stuff, share annot over threads">attachment 73820</a> <a href="attachment.cgi?id=73820&action=edit" title="fix font info stuff, share annot over threads">[details]</a></span> <a href='page.cgi?id=splinter.html&bug=59927&attachment=73820'>[review]</a>
fix font info stuff, share annot over threads
(In reply to <a href="show_bug.cgi?id=59927#c5">comment #5</a>)
<span class="quote">> Personally i don't think the define makes sense, just depend on the poppler
> version and that's it.</span >
Ok, I removed it from this patch
<span class="quote">> There's on true that shold be gTrue to be pedantically correct</span >
Ok, I changed it. But to be pedantically, too, I only correct MY changes, NOT
the other uses of true/false :-)
<span class="quote">> Why the setoverprint code move? (I think setoverprint should be also gone
> from globalparams and move to the outputdev itself, this way we can have one
> with overprint and one without overprint with no fear that it'll collide one
> with eachother, but that's for the future :D)</span >
Future is there, done.
!!!!!!!!!!!!WARNING!!!!!!!!!!!!
Running Adam's test, I encounterd that there is an "unlock" missing in XRef.cc.
This can cause deadlocks with the actual git master, so this part of the patch
should be committed asap to avoid bug reports. It will be solved in better way
in <a class="bz_bug_link
bz_status_NEW "
title="NEW --- - Cleanup patch for bug 50992"
href="show_bug.cgi?id=59933">bug 59933</a>, but I stopped working on it because I think we should close this
bug first!
Another comment to this patch: I insert some locks/unlocks in the Annot stuff,
because Adam's test case shares annotations over threads and can modify an
annotation at the same time in differnt threads. I don't think that this is a
normal use case, but nevertheless, better to have it!</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>