Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Relax increment/decrement rule. #16

Merged
merged 1 commit into from Feb 24, 2013

Conversation

Projects
None yet
2 participants
Contributor

eddieajau commented Dec 17, 2012

The increment/decrement rule needs to be relaxed because it's saying that $x += 1 should be replaced with $x++. The develop may intend to make the code more readable.

Citing references for this:

http://stackoverflow.com/questions/971312/why-avoid-increment-and-decrement-operators-in-javascript

While there may be valid arguments for using increment and decrement as a practice, forcing code to use them should not be part of the coding standard.

dongilbert added a commit that referenced this pull request Feb 24, 2013

Merge pull request #16 from eddieajau/rules
Relax increment/decrement rule.

@dongilbert dongilbert merged commit 34e52d6 into joomla:master Feb 24, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment