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

Not all command line arguments seem to be processed #39

Open
szaydel opened this issue Apr 20, 2018 · 0 comments
Open

Not all command line arguments seem to be processed #39

szaydel opened this issue Apr 20, 2018 · 0 comments

Comments

@szaydel
Copy link
Contributor

szaydel commented Apr 20, 2018

Hi, I noticed that depending upon what I pass to C sometimes result is not what I expect. Having looked at https://github.com/ryanmjacobs/c/blob/master/c#L54 I am a bit confused and thought it is worth raising this question. I see that here instead of trying to process all command line arguments, we there is a for arg in $1, which even if we do a shift later won't mean that this for look is going to run more than once. I assumed that maybe that should have been $@, but it would break some of this logic also.

Anyway, I think my question, since there are not comments there to help me understand, is whether or not this loop at line 54 was meant to be gone through more than once. If so, I think there may be problems with it.

Thanks!

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

1 participant