Re: [Codelite-developers] cppchecker settings

[Eran I. <era...@gm...>]

See below

On Fri, Sep 25, 2009 at 3:05 PM, Eran Ifrah <era...@gm...> wrote:

>
>
> On Fri, Sep 25, 2009 at 3:01 PM, <da...@4p...> wrote:
>
>> Hi Eran,
>>
>> I've been playing with the new cppchecker settings plugin. It's
>> certainly a worthy addition to CL.
>>
>> However a couple of things:
>>  1) First, the Settings dialog is too small for
>> its text, at least for my font-size. The patch makes it
>> wxDefaultSize, which works for me. Please check on MSWin, though.
>>
>> I will
>
  2) The only way I could find to get it to run is the workspace/project
>>  context menu. Did I miss somewhere else? Should it also have a
>>  menu entry in Plugsins > CppCheck ?
>>
>> You can run it from the file explorer as well, which will allow you to run
> on it files as well
>
>
>>  3) Like other people, I'd welcome a prominent 'Done' message; perhaps
>>  at the end of the errors...
>>
>> Yea, I got many feedbacks about it, I will add one :)
>

I added a 'Done' message when the processing is completed


>
>
>>  4) It would be nice if the results were cleared on closing the
>>  workspace. At the moment they're still there when a new workspace is
>>  loaded. However I can't see any easy way to call
>>  CppCheckReportPage::Clear from the frame/manager. Do you think it's
>>  worth storing a pointer there?
>>
>> No, it is better to handle the plugin's event "wxEVT_WORKSPACE_CLOSED"
>

Done, the CppCheck plugin now handles the wxEVT_WORKSPACE_CLOSED event and
clears the view once the workspace is closed


>
>
>> The good news is that it can't find any significant errors in 4Pane :)
>> The bad news is that it doesn't like my coding style :(
>>
>> Same here, it did detected 1 memory leak (which is indeed a memory leak)
> but it cant handle the wxWidgets way of deleting windows...
> In addition, it keep complains about ++iter vs iter++
>
>
>> Best wishes,
>>
>> David
>
>
>
>
> --
> Eran Ifrah
> Cross platform, open source C++ IDE: http://www.codelite.org
>
>


-- 
Eran Ifrah
Cross platform, open source C++ IDE: http://www.codelite.org