Splitting debugger in two - specific debugger and common GUI

[ollydbg]

Hi, all debugger exports, did you consider saving the breakpoint after we close a project. and load the breakpoint when loading a project?

I have done that several months ago, I nearly forget that. see here: Watches and breakpoints persistent, I'm not sure this can be easily done in the debugger branch.

[oBFusCATed]

Morten: I've updated the svn server, probably I need to do something with the repo, will look at it tonight

Olly: I've not forgotten this feature, but I don't like the implementation! C::B needs an API to store user information per project/workspace in single file, not N files as it is now.

[MortenMacFly]

see here: Watches and breakpoints persistent, I'm not sure this can be easily done in the debugger branch.

I still have that merged (as far as possible) with the debugger branch in my local copy. It's not forgotten, just pending as it needs some re-write.

[oBFusCATed]

Mismatched FS module version for 'fsfs': Found 1.6.6 expected 1.6.9.

My svn server needed a restart after the update, should work now 

[MortenMacFly]

should work now 

Yes, it's back again.

[oBFusCATed]

And another patch: http://smrt.is-a-geek.org/codeblocks/dbg_refactor0012.patch

1. Implemented Set Next statement/Jump to cursor, yey
2. Fixed a bug in the code for adding breakpoints
3. Made the watches window more user friendly
4. Fixed a redraw issue in watches window

Morten: Do you have a pause icon in the same style as the other icons in the debugger set?

[Ceniza]

1. Implemented Set Next statement/Jump to cursor, yey

That sounds really nice

[blueshake]

hope it can be merged into trunk as soon as possible.

[oBFusCATed]

You're welcome to test it  :lol: 

[oBFusCATed]

Little addition to the previous patch: http://smrt.is-a-geek.org/codeblocks/dbg_refactor0012.1.patch

1. Implemented Set Next statement/Jump to cursor, yey
2. Fixed a bug in the code for adding breakpoints
3. Made the watches window more user friendly
4. Fixed a redraw issue in watches window
New: 5.  Fixed bug: "adding breakpoint while running doesn't hide the current position marker"

[MortenMacFly]

Morten: Do you have a pause icon in the same style as the other icons in the debugger set?

No, but I think it should be orange and have the typical pause sign as you know it from CD players, like:

[ " ]

:lol:

[oBFusCATed]

Agreed  :lol: now we need someone to do it
Where have you got the other icons in the new set?

[MortenMacFly]

Where have you got the other icons in the new set?

I've forgotten. It was either from a patch (search for closed patches assigned to me in the patch tracker) or (even worse) from the forums. But I think we should start with re-colouring the existing icon from green to orange... this can be easily done.- ;-)

[oBFusCATed]

OK, I'll see what I can do.

Another question:
What do you think is needed in order to get this branch merged in trunk?
The major feature that is missing (user visible) is "local variables/function arguments", but for that one I'll need to modify C::B a lot, because I want them to be in separate windows, not as before in the watches window.

[oBFusCATed]

OK, I have an icon:

22x22: http://smrt.is-a-geek.org/codeblocks/new_icon/dbgpause.png
16x16: http://smrt.is-a-geek.org/codeblocks/new_icon/16x16/dbgpause.png
patch: http://smrt.is-a-geek.org/codeblocks/new_icon/new_icon.patch

p.s. the icons look very bad with a darker theme.

http://smrt.is-a-geek.org/codeblocks/new_icon/icon_bug.png // notice the halos on the Run, Build And Run and Start/Continue icons

Would you accept a fix for them ?