search

processing / processing-docs

[Deprecated] Processing reference, examples, tutorials, and website
https://processing.org
371 stars 179 forks source link

Explanation of `+=` #847

Closed soegaard closed 1 year ago

soegaard commented 3 years ago

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 commented 1 year ago

@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 commented 1 year ago

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

soegaard commented 1 year ago

@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 commented 1 year ago

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.