?I am not sure if I am answering correctly (because I usually do not use
mailing lists, I just selected the "Answer to all" button of my mail
program), but I hope so.
I now checked the code again, but with a newer version of cppcheck. The
results are in the attachement.
If you would like to try to check the code by yourself, see
https://sourceforge.net/projects/cppcheck/ (the tool is very easy to use)
-----Ursprüngliche Nachricht-----
From: Hyrum K. Wright
Sent: Thursday, October 07, 2010 2:56 PM
To: Philipp Kloke
Cc: users_at_subversion.apache.org
Subject: Re: Checked Subversions code with cppcheck
On Mon, Oct 4, 2010 at 9:59 AM, Hyrum K. Wright
<hyrum_wright_at_mail.utexas.edu> wrote:
> On Mon, Oct 4, 2010 at 9:34 AM, Philipp Kloke <philipp.kloke_at_web.de>
> wrote:
>> ?Hi,
>>
>> I used cppcheck to analyze the source code of Subversion.
>> Cppcheck claims to have found five errors and some style problems.
>>
>> Please have a look on the result file in the attachement.
>
> Thanks for the feedback! I'll look at making some of these changes,
> and let you know how it goes. (In the future, as this is related to
> the development of Subversion, feel free to send this type of mail
> straight to dev_at_subversion.apache.org.)
I've incorporated as many fixes as I could, given the drift on trunk.
Could you rerun your tool and report the result back?
Thanks,
-Hyrum
Received on 2010-10-07 17:05:15 CEST