As discussed, I did the pre-merge cleanup, except:
- set_debugger() should probably not be an efun.
Apparently it has to be, else it appears as an unknown identifier in master.pike.
- Move low_enable_breakpoint and low_disable_breakpoint to program.c
I didn't really see any point in this, since these are helper methods of the debug_breakpoint pikeclass that change it's state.
I will go further with the TODOs, but I would feel much more comfortable if the changes in src and master were on master branch, since it would spare me regular sync nightmares like merge conflicts, etc. So if there is anything else to correct, please let me know. Otherwise - feel free to merge the changes.
They can be found on branch mkrawczuk_debugger_refactor. By the way, I was really unhappy about breaking the branch naming convention. The pre-receive hook kept telling me that "Common topic branch names are not allowed to contain /.", and I wanted to name my branch "mkrawczuk/debugger_refactor".
Again, thanks for a great conference and hoping to see you soon.