Re: Checked Subversions code with cppcheck

2010-10-07 Thread Hyrum K. Wright
On Thu, Oct 7, 2010 at 3:06 PM, Itamar O wrote: > On Thu, Oct 7, 2010 at 5:04 PM, Philipp Kloke wrote: >> >> ?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 che

Re: Checked Subversions code with cppcheck

2010-10-07 Thread Itamar O
On Thu, Oct 7, 2010 at 5:04 PM, Philipp Kloke wrote: > ?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.

Re: Checked Subversions code with cppcheck

2010-10-07 Thread Philipp Kloke
ubversions code with cppcheck On Mon, Oct 4, 2010 at 9:59 AM, Hyrum K. Wright wrote: On Mon, Oct 4, 2010 at 9:34 AM, Philipp Kloke 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

Re: Checked Subversions code with cppcheck

2010-10-07 Thread Hyrum K. Wright
On Mon, Oct 4, 2010 at 9:59 AM, Hyrum K. Wright wrote: > On Mon, Oct 4, 2010 at 9:34 AM, Philipp Kloke 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 i

Re: Checked Subversions code with cppcheck

2010-10-04 Thread Hyrum K. Wright
On Mon, Oct 4, 2010 at 9:34 AM, Philipp Kloke 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 makin

Checked Subversions code with cppcheck

2010-10-04 Thread Philipp Kloke
?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. [ClientContext.cpp:40]: (style) Member variable not initialized in the constructor 'ClientContext::m_cancelO