patch for todo plug-in

[spedgenius]

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]

[MortenMacFly]

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.

[spedgenius]

I will work on merging

[spedgenius]

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

[attachment deleted by admin]

[Cryogen]

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 Select Expand

    m_pListLog->m_pAllowedTypesDlg->Clear();

before

Code Select Expand

    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.next.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]

[spedgenius]

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

[Cryogen]

No problem. Happy to help.