Skip to content

implementing ASM window#222

Merged
nakst merged 2 commits into
nakst:masterfrom
LinArcX:implementingAsmWindow
Jun 16, 2026
Merged

implementing ASM window#222
nakst merged 2 commits into
nakst:masterfrom
LinArcX:implementingAsmWindow

Conversation

@LinArcX

@LinArcX LinArcX commented Jun 7, 2026

Copy link
Copy Markdown
Contributor

This PR gives you the ability to see assembly instructions in a separate window, while you're debugging your source code in the main window:

asmFinal

@LinArcX LinArcX force-pushed the implementingAsmWindow branch from 5132bb5 to 9fc4402 Compare June 7, 2026 22:15
@LinArcX LinArcX marked this pull request as ready for review June 7, 2026 22:20
Comment thread windows.cpp Outdated
for (char *p = evaluateResult; p < current; p++) {
if (*p == '\n') line++;
}
asmCode->vScroll->position = line * lineHeight - (viewHeight / 2);

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does UICodeFocusLine work here?

@LinArcX LinArcX Jun 13, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi, i tried, but it didn't work. (vScroll position is always at the bottom of the ASM window)

Comment thread windows.cpp Outdated
}

void ASMWindowUpdate(const char *data, UIElement *element)
{

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry to nitpick but in our coding styling the { goes on the above line.

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also we use tabs over spaces.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

both fixed.

@nakst nakst merged commit ed31b8b into nakst:master Jun 16, 2026
1 check passed
@nakst

nakst commented Jun 16, 2026

Copy link
Copy Markdown
Owner

Thank you for the commit!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants