issue-#142 adding a disassembler to the ocf importer

[Josef Templ]

I have fixed the utf-8 conversion for procedure names.

For the diffs see http://redmine.blackboxframework.org/pr ... 38f8d9c26c.

I also removed unused constants in DevBrowser from issue-#140 that happend to
reappear after a manual merge conflict resolution.
(I also removed a couple of commented debug statements.)

I did not have time to look closely into the fixup chains.
(My first impression is that it is not easy to make the decoder
a symbolic disassembler that outputs symbolic names where possible.
It is also not easy to identify the places that may benefit from a comment like "fixup".
There is already a lot of symbolic information by means of merging the decoded instructions
with the source code. For understanding the output of the compiler's code generator this
is sufficient for me.)

- Josef




- Josef

[Ivan Denisov]

Checked this version:
http://blackboxframework.org/unstable/i ... a1.751.zip

Utf8 procedures are shown correctly now.

[Ivan Denisov]

Instructions that need fixups can not be displayed "correctly" because the fixups are done
after loading the module. They refer to addresses only known at runtime, that's why
they need to be "fixed up" by the loader.
The only thing that may be possible in some cases is to add the target as a comment.
With the source merge feature, however, you see the target anyway from the source.

Can you please add this comment, that "this address is shifted by the fixup during the module load"? Or something like this...

[Ivan Denisov]

Josef, do you think that this address problem does not require comments? There should be some documentation explanation anyway. DevDecoder386 does not have any documentation now!

[Josef Templ]

The docu could be added to DevBrowser.ImportCodeFile.
BTW, the docu of DevBrowser.ShowCodeFile is inconsistent because ShowCodeFile
does not support filtering when a qualified identifier is selected.

By adding the docu to DevDecoder386 it will be hard to find for users.
However, a DevDecoder386 docu stub for "internal module" should be added indeed.

- Josef

[Josef Templ]

The docu has been improved as proposed by Ivan.
Please review.

see changes at https://redmine.blackboxframework.org/p ... b291b9b0fb

- Josef

[Robert]

It looks like we should vote on incorporating these changes.

[Ivan Denisov]

It looks like we should vote on incorporating these changes.

Agree. With documentation it looks finished.

[Robert]

Does it make sense to vote on an issue that is not open? Do we need a new issue, or can this be reopened?

I'm not sure of the process here.

[Ivan Denisov]

Does it make sense to vote on an issue that is not open? Do we need a new issue, or can this be reopened?

I'm not sure of the process here.

I think, we can reopen it before voting.

I changed status to resolved: https://redmine.blackboxframework.org/issues/142