Author Topic: Patch for C::B codecomplete bug needs attention  (Read 4794 times)

Offline Greatwolf

  • Multiple posting newcomer
  • *
  • Posts: 48
Patch for C::B codecomplete bug needs attention
« on: December 17, 2010, 02:54:14 am »
Hi,

This post is really just to redirect some attention to my original post here. ollydbg was the only one to really respond to my post and he even posted a patch that fixes this bug (many thanks btw kudos to him!). Can one of the maintainers with commit access please review this and apply the patch to the trunk so this bug is fixed for the next version?

Many thanks

Offline ollydbg

  • Developer
  • Lives here!
  • *****
  • Posts: 6077
  • OpenCV and Robotics
    • Chinese OpenCV forum moderator
Re: Patch for C::B codecomplete bug needs attention
« Reply #1 on: December 17, 2010, 03:54:16 am »
 :D
my patch should be reviewed.
If some piece of memory should be reused, turn them to variables (or const variables).
If some piece of operations should be reused, turn them to functions.
If they happened together, then turn them to classes.

Offline MortenMacFly

  • Administrator
  • Lives here!
  • *****
  • Posts: 9723
Re: Patch for C::B codecomplete bug needs attention
« Reply #2 on: December 18, 2010, 01:49:35 pm »
my patch should be reviewed.
I've at least applied it. However, I am rarely using C::B these days (temporarily!) so I cannot do intensive testing. But it shouldn't get lost, at least...
Compiler logging: Settings->Compiler & Debugger->tab "Other"->Compiler logging="Full command line"
C::B Manual: https://www.codeblocks.org/docs/main_codeblocks_en.html
C::B FAQ: https://wiki.codeblocks.org/index.php?title=FAQ

Offline Loaden

  • Lives here!
  • ****
  • Posts: 1014
Re: Patch for C::B codecomplete bug needs attention
« Reply #3 on: December 20, 2010, 03:17:59 am »
After testing, this patch can fix the issue.
Thanks!

Offline ollydbg

  • Developer
  • Lives here!
  • *****
  • Posts: 6077
  • OpenCV and Robotics
    • Chinese OpenCV forum moderator
Re: Patch for C::B codecomplete bug needs attention
« Reply #4 on: December 20, 2010, 03:21:54 am »
After testing, this patch can fix the issue.
Thanks!
not sure this patch may cause other issue.... :D
If some piece of memory should be reused, turn them to variables (or const variables).
If some piece of operations should be reused, turn them to functions.
If they happened together, then turn them to classes.