News:

As usual while waiting for the next release - don't forget to check the nightly builds in the forum.

Main Menu

CC toolbar enhancement

Started by sbezgodov, March 18, 2010, 01:37:22 PM

Previous topic - Next topic
Print
Go Down Pages 1 2 All
User actions

ollydbg

#15
March 25, 2010, 06:45:25 AM
@sbezgodov

I have blame the "token.cpp", and it seems these code snippet exists from rev 1711 by rickg22 in 2006-1-11. Maybe, rickg22 can answer your question. But now, he is not so active in this forum. :(

I have read these code, and can't find any reason we don't apply your patch on token.cpp.  :D

Any comments from morten or jens or rickg22?

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.

MortenMacFly

#16
March 25, 2010, 08:29:13 AM
Quote from: ollydbg on March 25, 2010, 06:45:25 AM
I have read these code, and can't find any reason we don't apply your patch on token.cpp.  :D
Any comments from morten or jens or rickg22?
I can't either, but it's on freeze atm..
Compiler logging: Settings->Compiler & Debugger->tab "Other"->Compiler logging="Full command line"
C::B Manual: [url="https://www.codeblocks.org/docs/main_codeblocks_en.html"]https://www.codeblocks.org/docs/main_codeblocks_en.html[/url]
C::B FAQ: [url="https://wiki.codeblocks.org/index.php?title=FAQ"]https://wiki.codeblocks.org/index.php?title=FAQ[/url]

ptDev

#17
March 25, 2010, 11:27:13 AM
While I am not in the developing team, I think you could let this one slip...
After all, it is correcting an obvious omission from the UI (the permanently disabled listbox), and in that sense, it is a bug fix...

Jenna

#18
March 26, 2010, 10:31:14 AM
Quote from: ptDev on March 25, 2010, 11:27:13 AM
While I am not in the developing team, I think you could let this one slip...
After all, it is correcting an obvious omission from the UI (the permanently disabled listbox), and in that sense, it is a bug fix...
But it needs much more testing than a 3-liner bug-fix test, because it does change features of the IDE, even if it would be nice to have this working.

I don't think there should be an exception for this.

Loaden

#19
April 21, 2010, 05:43:27 PM Last Edit: April 22, 2010, 06:37:29 AM by Loaden
Hi, I make a little modify, to support global namespace "::".
And it's can be Disable / Enable switch.
Based SVN6204.

[attachment deleted by admin]

Loaden

#20
May 02, 2010, 05:53:32 AM
Hi, sbezgodov, this patch not work in linux.
Can you check it?
Print
Go Up Pages 1 2 All
User actions