Skip to content
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

linux uppercaser not uppercaser #77

Open
yancyribbens opened this issue Aug 7, 2022 · 2 comments
Open

linux uppercaser not uppercaser #77

yancyribbens opened this issue Aug 7, 2022 · 2 comments

Comments

@yancyribbens
Copy link
Contributor

yancyribbens commented Aug 7, 2022

I tested the example file uppercaser-linux-c.s although it doesn't look like it's designed to uppercaser..

from the comment section in the file uppercaser-linux-c.s

# Will print out command-line arguments and argument count

Also, it seems like while it's designed to print the number of command line arguments, it always prints -1 for every amount of arguments I feed it..

@jessicard
Copy link
Contributor

Oh no! Thank you for catching this. Unfortunately I do not have a computer that was able to test this file when it came in, so I was unable to fully verify it working.

Some solutions:

  • Would you have the time to potentially fix it?
  • We could fix the arg count and rename the file to what it's actually doing
  • We can remove the file entirely (not ideal - I think any working code is useful code for Assembly stuff)

Thoughts @yancyribbens?

@TheOneKevin
Copy link
Contributor

Hi! Could you drop the platform and compiler you used? I remember this file doesn't perform any actual uppercasing but the it should at least print the arguments. I will try to reproduce this as soon as I have time.

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

No branches or pull requests

3 participants