Code::Blocks Forums

Developer forums (C::B DEVELOPMENT STRICTLY!) => Plugins development => Topic started by: spedgenius on December 23, 2009, 09:06:13 am

Title: patch for todo plug-in
Post by: spedgenius on December 23, 2009, 09:06:13 am

I have a patch for the todo plug-in.  it searches faster over large projects, is less buggy, and allows for choosing which type of note to look for between refreshes. if anyone wants the prebuilt dll. just let me know where i can post it since it is to large for this board.

[attachment deleted by admin]

Title: Re: patch for todo plug-in
Post by: MortenMacFly on December 23, 2009, 07:38:48 pm

Quote from: spedgenius on December 23, 2009, 09:06:13 am

I have a patch for the todo plug-in.  it searches faster over large projects, is less buggy, and allows for choosing which type of note to look for between refreshes. if anyone wants the prebuilt dll. just let me know where i can post it since it is to large for this board.

This patch does not apply on trunk. I saw that your base version is 5201 which is *very* old (we are at 5986 meanwhile!!! :shock:).

Please provide a patch based on trunk, otherwise it's really getting hard. Alterantively you can try to provide the complete sources of the plugin but I am not sure if it's getting easier then.

Bear in mind that any binary based on revision 5201 will *not* work with recent versions of C::B.

Title: Re: patch for todo plug-in
Post by: spedgenius on December 24, 2009, 02:44:46 am

I will work on merging

Title: Re: patch for todo plug-in
Post by: spedgenius on December 24, 2009, 03:53:27 am

alright here is the new patch, let me know what your thoughts are.

[attachment deleted by admin]

Title: Re: patch for todo plug-in
Post by: Cryogen on March 20, 2010, 05:41:54 am

 Thanks for this great feature. Very nice. I found a bug, though, that causes the type dialog to show another copy of all of the types after each comment is entered. This happens because OnAddItem() calls LoadTypes() each time and LoadTypes() simply adds all of the types to the existing collection. The solution is to insert

Code

    m_pListLog->m_pAllowedTypesDlg->Clear();

before

Code

    m_pListLog->m_pAllowedTypesDlg->AddItem(m_Types);

which is in LoadTypes() at line 244 of todolist.cpp in my environment.

I like the feature so much that I have merged it with my patch that adds a C-style comment that works on doxygen comment blocks, since they conflicted. Your change detects my comments nicely. I also resized the dialogue slightly so that you don't have to scroll to get to the bottom items. Details of my patch are at http://forums.codeblocks.org/index.php/topic,12130.0.html. I updated my patch on Berlios with the merged version, too. It's patch 2941 at https://developer.berlios.de/patch/index.php. Copy attached.

Cheers,

  Cryo.





[attachment deleted by admin]

Title: Re: patch for todo plug-in
Post by: spedgenius on March 22, 2010, 10:50:30 pm

thanks for the feedback cryogen,  Ill be sure to change that in my source.  I always forget to do that with list controls.

Title: Re: patch for todo plug-in
Post by: Cryogen on March 26, 2010, 01:52:19 am

No problem. Happy to help.  :)