Skip to content

Fix spelling errors. Change the output message to be the same in every instance.#88

Merged
klaudiagrz merged 1 commit into
0xAX:masterfrom
awatne:asm3/spelling_errors
May 19, 2026
Merged

Fix spelling errors. Change the output message to be the same in every instance.#88
klaudiagrz merged 1 commit into
0xAX:masterfrom
awatne:asm3/spelling_errors

Conversation

@awatne
Copy link
Copy Markdown
Contributor

@awatne awatne commented May 19, 2026

Hello.
I went through the asm_3 post and found a couple spelling errors. I also noticed that the output message wasn't the same in every instance, so I changed that too. This is how it appears the first time:

    WRONG_ARGC_MSG  db "Error: expected two command-line arguments", 10
    ;; Length of the WRONG_ARGC_MSG message
    WRONG_ARGC_MSG_LEN equ 43

I also saw that you use both 10 and 0xa for newline, but i didn't change that. I thought it was informative.

@awatne awatne requested review from 0xAX and klaudiagrz as code owners May 19, 2026 13:22
@klaudiagrz
Copy link
Copy Markdown
Collaborator

Hi @awatne ! 👋🏻

Thank you for the contribution, nice findings!

@klaudiagrz klaudiagrz merged commit 60c03bb into 0xAX:master May 19, 2026
1 check passed
@awatne awatne deleted the asm3/spelling_errors branch May 19, 2026 15:46
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