Author Topic: patch for todo plug-in  (Read 6140 times)

Offline spedgenius

  • Single posting newcomer
  • *
  • Posts: 8
patch for todo plug-in
« 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]

Offline MortenMacFly

  • Administrator
  • Lives here!
  • *****
  • Posts: 9508
Re: patch for todo plug-in
« Reply #1 on: December 23, 2009, 07:38:48 pm »
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.
« Last Edit: December 23, 2009, 07:41:42 pm by MortenMacFly »
Compiler logging: Settings->Compiler & Debugger->tab "Other"->Compiler logging="Full command line"
C::B Manual: http://www.codeblocks.org/docs/main_codeblocks_en.html
C::B FAQ: http://wiki.codeblocks.org/index.php?title=FAQ

Offline spedgenius

  • Single posting newcomer
  • *
  • Posts: 8
Re: patch for todo plug-in
« Reply #2 on: December 24, 2009, 02:44:46 am »
I will work on merging

Offline spedgenius

  • Single posting newcomer
  • *
  • Posts: 8
Re: patch for todo plug-in
« Reply #3 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]

Offline Cryogen

  • Regular
  • ***
  • Posts: 260
Re: patch for todo plug-in
« Reply #4 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: [Select]
    m_pListLog->m_pAllowedTypesDlg->Clear();
before

Code: [Select]
    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]

Offline spedgenius

  • Single posting newcomer
  • *
  • Posts: 8
Re: patch for todo plug-in
« Reply #5 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.

Offline Cryogen

  • Regular
  • ***
  • Posts: 260
Re: patch for todo plug-in
« Reply #6 on: March 26, 2010, 01:52:19 am »
No problem. Happy to help.  :)