Assignment should be wrapped up in parentheses
It's more clearly to use braces instead of &&
Use tab characters instead of spaces
Merge pull request #5367 from micate/4.2

Use if instead of && for assignment