Skip to content

int() works on String #806

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

Closed
edringel opened this issue Mar 28, 2020 · 4 comments
Closed

int() works on String #806

edringel opened this issue Mar 28, 2020 · 4 comments

Comments

@edringel
Copy link

Documentation does not indicate that int() will perform correctly on a String variable, which it does.

URL(s) of affected page(s)

https://processing.org/reference/intconvert_.html

Proposed fix

change the following

Description | Converts any value of a primitive data type (boolean, byte, char, color, float, int, or long) to its integer representation.When an array of values is passed in, then an int array of the same length is returned.

to the following

Description | Converts any value of a primitive data type (boolean, byte, char, color, float, int, or long) or String to its integer representation.When an array of values is passed in, then an int array of the same length is returned.

@khan-rehan
Copy link

can I do this?

@edringel
Copy link
Author

So, I don't understand the process or etiquette involved here. As far as I am concerned, if you have the rights and authority to make the change it would be good. Thank you

@jeremydouglass
Copy link
Contributor

@khan-rehan -- go ahead and create a PR referencing this issue and @prisonerjohn or @REAS will merge it (at least, I can't see any reason why they wouldn't in this case!). Be sure to make your PR editable.

Also, be patient, please -- PRs on docs are reviewed and merged in batches, so they may be handled immediately or they may take a month or two. My last few docs PRs averaged about six weeks.

@REAS
Copy link
Member

REAS commented Jun 3, 2020

Thank you. I've updated this. It will be online soon.

@REAS REAS closed this as completed Jun 3, 2020
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

4 participants