On So  6.Dez'09 at  3:08:34 +0100, Nicolas Bonifas wrote:
> 
> I found the problems that I corrected in my two previous patches
> using the nice cppcheck tool (cppcheck.sf.net).

Then it would be even more helpfull if you included the relevant
output of 'cppcheck' in the log. That may help review the patch
and understand the motivation.

Another tool which I was using the other day was 'sparse'. That
is also a good source of motivation for cleanups.

> It also reported that some functions are never called. I'm sure
> Carlos will be delighted to remove those of them which may never be
> of any use (probably all of them). 

Heh, removing unused stuff from the wmaker I use is nice. So thanks
about that, I will take a look at those functions and probably 
remove them.

Thanks Nicolas, your help in cleaning up things is much appreciated!


-- 
To unsubscribe, send mail to [email protected].

Reply via email to