Skip to content

Minor change to example sketch comment #5407

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

Merged
merged 1 commit into from
Sep 23, 2016
Merged

Conversation

pfeerick
Copy link
Contributor

Can't be the right word! ;)

Can't be the right word! ;)
@pfeerick pfeerick changed the title Minor change to example comment Minor change to example sketch comment Sep 23, 2016
@rkost
Copy link

rkost commented Sep 23, 2016

I think 'can' is the right word here. You need to read it as conclusion. The number is to big than to store it to an integer variable. One may say '... so it is not possible to store in an int' or something like this

@facchinm
Copy link
Member

Wording is correct indeed 😄

@facchinm facchinm closed this Sep 23, 2016
@pfeerick
Copy link
Contributor Author

@rkost Indeed... hence why I removed the 't... as it was originally "can't" ... as "become a bigger number than can not be stored in an int" makes no grammatical sense! @facchinm might want to come back at apply this at some point ;)

@facchinm facchinm reopened this Sep 23, 2016
@facchinm
Copy link
Member

Err, it turned out that in Italy we have some problems with double negatives 😄
After a discussion with the best linguists it turned out that you are right, so I'm merging this 😉 Sorry for closing too soon

@facchinm facchinm merged commit be9bbc3 into arduino:master Sep 23, 2016
@pfeerick
Copy link
Contributor Author

No worries... I probably didn't help with my joke about "can't"... I did the assume thing, thinking the commit would speak for itself... 😵

@pfeerick pfeerick deleted the patch-1 branch September 24, 2016 00:34
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.

3 participants