-
Notifications
You must be signed in to change notification settings - Fork 16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Disassembled RBRA looks odd on VGA #132
Comments
When I extended the assembler for \t in ASCII strings I went a not too-clever route to implement this feature which most probably is the reason for this ugly output (lines 686 - 689 in qasm.c). A \t control sequence basically gets replaced by a SPACE (ugly, it was a pretty quick fix) and an ASCII 9, i.e. a "real" tab control character. I assume that this is displayed as a thick vertical bar in the output. The VGA output routines should take care of tab control characters. Regarding spacing: The standard is a tab stop every eight columns. We should keep that - maybe in a configurable way, what do you think? |
OK got it - then we will enhance the VGA routines in the monitor. Just out of curiosity: why did you add the space and not just use the "pure" tab character? @MJoergen should I assign this one to you or me? |
The reason for 0x0a 0x09 is simple: Because I did not find a clean way to do so. ;-) If you can fix this easily, please do so. :-) |
@sy2002 . I think it is faster if you assign it to yourself :-) |
OK. Done :-) |
I just changed the assembler (its gets more and more ugly... ARGL) so that \t is now translated into 0x09 instead of 0x09 0x20 as before. Thus we got rid of the erroneous blank character. |
Hooraaay :-) |
On VGA, the Monitor's new feature to disassemble
RBRA
in a convenient way by also showing the absolute target address looks odd on VGA. I guess is due to the usage of tabs. See image:There are multiple ways to solve that:
\t
in the Monitor's VGA output routines. (We would need to decide how many spaces)Up to the group to discuss how to proceed here.
The text was updated successfully, but these errors were encountered: