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

Rename variable: 'byte' is a reserved word. #76

Merged
merged 1 commit into from Jan 15, 2014

Conversation

Projects
None yet
2 participants
@sergio91pt
Contributor

sergio91pt commented Nov 24, 2013

No description provided.

@dduponchel

This comment has been minimized.

Show comment
Hide comment
@dduponchel

dduponchel Nov 24, 2013

Collaborator

Nice catch ! For future references, I found byte in the 3rd edition (7.5.3, Future Reserved Words) but not in the 5.1 Edition.
I will merge this pull request as soon as #74 is merged (it contains a lot of changes and I don't want to generate merge conflicts). Thanks for the bug report :-)

Collaborator

dduponchel commented Nov 24, 2013

Nice catch ! For future references, I found byte in the 3rd edition (7.5.3, Future Reserved Words) but not in the 5.1 Edition.
I will merge this pull request as soon as #74 is merged (it contains a lot of changes and I don't want to generate merge conflicts). Thanks for the bug report :-)

@dduponchel dduponchel merged commit 8e468ee into Stuk:master Jan 15, 2014

1 check failed

default The Travis CI build failed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment