[Libreoffice] [PATCH] Things by cppcheck - [base] repo

Thorsten Behrens thb at documentfoundation.org
Mon Oct 11 05:59:25 PDT 2010


Kayo Hamid wrote:
> I'm sending this patch for review because i do not know if these changes are ok.
> The patch covers variables assigned but not used and scope of variables inside if/else/loops.
> revol_
> 
Hi revol_,

hm, that patch breaks building dbaccess in several ways - at least
pDesc, nNumberFormat, fOutNumber etc. are needed where they have
been originally. Could you have a second look pleae?

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/20101011/57a81ca2/attachment.pgp>


More information about the LibreOffice mailing list