[Libreoffice] [PUSHED] [PATCHES] Fixed some CLang++ errors/warnings in sal

Thorsten Behrens thb at documentfoundation.org
Thu Apr 14 03:33:26 PDT 2011


Julien Chaffraix wrote:
> Subject: [PATCH 1/5] Added error handling for pthread_mutexattr_settype.
> 
Handled slightly different, to avoid duplication & retain the
pthread_mutexattr_destroy call.

> Subject: [PATCH 2/5] Added handling for the write errors in receiveFdPipe.
> 
Applied - would you want to look into the TODO case as well? A rather 
nasty cause for errors ...

> Subject: [PATCH 3/5] Fixed a potential null-dereferencing error in osl_closeProfile
> Subject: [PATCH 4/5] Fixed some false positives 'dead assignments' seen in CLang++
> Subject: [PATCH 5/5] No need to check out execv return value.
> 
Applied - the extra ifs in 4/5 are a lesser evil than those warns, I
agree.

Many thanks for your fixes - pushed to the -3-4 branch, you're
fixing real bugs here.

Cheers,

-- Thorsten
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/libreoffice/attachments/20110414/db4d3f8b/attachment.pgp>


More information about the LibreOffice mailing list