[PATCH] Fix some errors detected with cppcheck

Aurimas Fišeras aurimas at members.fsf.org
Sat Aug 27 06:10:04 EDT 2011


2011.08.21 23:02, Christian Stimming rašė:
> Dear Aurimas,
>
> thank you very much for your patches. The code looks very good and those
> patches are surely an improvement. I've committed them all to SVN trunk.
>
> As John already said, submitting patches to bugzilla is another possibility.
> For single patches (especially non-trivial ones) we think this is the best way
> to handle them. However, downloading a series of patches from bugzilla
> attachments is somewhat inconvenient, so I also see some value in just sending
> them here as email attachments. Hm... I'm unsure whether it is important to
> state a general rule here. Whatever.
>
> Thanks for the code!
>

Hello,
I personally think that for trivial patches it is too much unnecessary 
work to submit them to bugzilla.

Here are some more patches fixing errors detected by cppcheck.

Aurimas
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0007-cppcheck-Fix-several-memory-leaks.patch
Type: text/x-patch
Size: 2727 bytes
Desc: not available
URL: <http://lists.gnucash.org/pipermail/gnucash-devel/attachments/20110827/8b76adb4/attachment-0005.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0006-cppcheck-Fix-mismatching-allocation-and-deallocation.patch
Type: text/x-patch
Size: 2640 bytes
Desc: not available
URL: <http://lists.gnucash.org/pipermail/gnucash-devel/attachments/20110827/8b76adb4/attachment-0006.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0005-cppcheck-Fix-a-possible-memory-leak-data.patch
Type: text/x-patch
Size: 1174 bytes
Desc: not available
URL: <http://lists.gnucash.org/pipermail/gnucash-devel/attachments/20110827/8b76adb4/attachment-0007.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0004-cppcheck-Fix-mismatching-allocation-and-deallocation.patch
Type: text/x-patch
Size: 1120 bytes
Desc: not available
URL: <http://lists.gnucash.org/pipermail/gnucash-devel/attachments/20110827/8b76adb4/attachment-0008.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0003-cppcheck-Remove-several-redundant-null-checks.patch
Type: text/x-patch
Size: 2380 bytes
Desc: not available
URL: <http://lists.gnucash.org/pipermail/gnucash-devel/attachments/20110827/8b76adb4/attachment-0009.bin>


More information about the gnucash-devel mailing list