To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=108914





------- Additional comments from h...@openoffice.org Thu Feb  4 09:55:35 +0000 
2010 -------
Except for a multi-threading problem with insufficient locking there is AFAIK 
no scenario which would 
change the mpGraphics inside an ImplNewFont(). There is no vdev created in any 
of the codepaths as 
the gcach_vdev was only used in the ancient SRD-codeline... RIP.

So, if there really is a multi-threading problem with insufficient locking then 
the patch only works 
around some bad aspect but there are countless other scenarios that could fail. 
Is it possible that you 
could put a hardware watchpoint on who sets the mpGraphics to NULL while we are 
inside the 
ImplNewFont() method?

The mentioned line "SalGraphics* pGraphics = mpGraphics" is ancient (<=Y2K). In 
my experience such 
a line is the result of "crunch mode before a release", where the changes are 
intented to be minimally-
invasive. The goal was probably to make a free-standing function a member of 
the OutputDevice class 
to give it access to its other members... Didn't do much more code archeology 
on that line since the 
pre-OOo repo is not easily available.

---------------------------------------------------------------------
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@gsl.openoffice.org
For additional commands, e-mail: issues-h...@gsl.openoffice.org


---------------------------------------------------------------------
To unsubscribe, e-mail: allbugs-unsubscr...@openoffice.org
For additional commands, e-mail: allbugs-h...@openoffice.org

Reply via email to