[Libreoffice] Mutex(Guard) ignores failure to create/acquire Mutex

Noel Grandin noel at peralex.com
Thu Aug 18 04:43:03 PDT 2011


Perhaps it would be a good idea to add some comments to that code so that analysis is preserving for future generations
of code archeologists :-)

Seriously, I often do this because it saves me time later when I look at some code and think "why is it doing _that__???"

Michael Meeks wrote:
> On Thu, 2011-08-18 at 01:57 -0500, Norbert Thiebaud wrote:
>> Guards are used and badly abused in the code, especially with the
>> infamous SolarMutex...
>> anything that would make that code slower or fatter than it already is
>> (with virtual functions and all), especially for corner-case
>> 'you-are-about-to-die-anyway' scenario, is bad.
> 	Right - throwing exceptions - or annotating methods as being able to
> throw exceptions for corner cases that are highly unlikely seems like
> just adding unwinding bloat for no major advantage.
>
>> so my opinion is, just let it be :-)
> 	Ditto; good catch though Lionel - great to have some scrutiny of this
> sort of issue: no doubt there are plenty of things that can fail that
> are not handled correctly currently, and would be good to improve.
> Thanks for caring !
>
> 	All the best,
>
> 		Michael.
>

Disclaimer: http://www.peralex.com/disclaimer.html


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/libreoffice/attachments/20110818/8a4c3517/attachment.htm>


More information about the LibreOffice mailing list