cbexaminer: I've looked at your patch, at last...
Here are some general suggestions:
1. Don't use abbreviations for the identifiers
2. Use camel-case for the identifiers: "m_myVar" instead of m_myvar
I've modified the patch a bit:
http://smrt.is-a-geek.org/codeblocks/patches/dbg/disasm_changes_mine.patch1. I've cleaned the code a bit, the behaviour should not be changed, please verify it.
2. Added a check if the disassembly dialog is visible in the step command base class.
The old patch was executing the disassembly command even if the dialog was closed.
More changes are required:
1. You should remove all the // TBD comments
2. You should improve the comments:
I can hardly follow them, because you've written some random thoughts or used too many words and I'm getting lost.
The comments should be short and clear.
3. A commit message will be needed - describe all the changes please...
Please do some cleaning and then I'll look at your patch again.
I've applied it in my local copy and I hope to do some development + testing tomorrow.