Skip to content

Explanation of += #847

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
soegaard opened this issue Jun 13, 2021 · 4 comments
Closed

Explanation of += #847

soegaard opened this issue Jun 13, 2021 · 4 comments
Assignees

Comments

@soegaard
Copy link

Issue description

The documentation of += states the syntax is:

value1 += value2
I suggest changing this to:

identifier += expression
or perhaps

variablename += expression

The problem is that identifiers (variablenames) aren't values.
Also the righthand side is not restricted to literals - the
right hand side can be any expression.

URL(s) of affected page(s)

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

Proposed fix

@soegaard
Copy link
Author

soegaard commented Jan 1, 2023

@Arya-Gupta
I don't have the "assign issue to someone" provilege, but since
the issue hasn't been touched since June, I don't think you
risk others working on it too.

@Arya-Gupta
Copy link

@soegaard So can I work on this issue now and submit a PR?

@soegaard
Copy link
Author

soegaard commented Jan 1, 2023

@Arya-Gupta

I don't speak for the Processing project, but yes.
In the end it will probably be @REAS or @shiffman that decides whether to merge your PR.

@SableRaf
Copy link
Contributor

SableRaf commented Jan 6, 2023

Thank you for your suggestion. We understand that the current syntax may not be entirely accurate, as identifiers and expressions are not technically considered "values". However, we believe that the current syntax is sufficient for beginners to understand the basic concept of the operator, which is to add a value to a variable.

Additionally, the use of more technical language such as "identifiers" and "expressions" may be confusing for beginners, who may not be familiar with these terms. We believe that the current syntax is more straightforward and easier to understand for the intended audience.

Therefore, we will be closing this issue and not implementing the proposed changes. However, we appreciate your feedback and will continue to review and improve the documentation as necessary. Thank you again for your suggestion.

Raphaël de Courville
Processing Community Lead

Note: This repository is being deprecated. If you'd like to raise other issues regarding the Processing documentation, we encourage you to submit an issue on the processing-website repository.

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