Closed ghost closed 12 years ago
None of these changes make any real difference; performance is the same, no bugs are fixed, no algorithms are changed, and the code is left no shorter (or only by 3 or 4 lines).
These changes are simply based on personal preference on where to put the last return, if there are multiple ones. In many cases, I prefer to put it in the else statement, so all returns are flush with each other (which makes it more clear for me).
For these reasons, I am not going to accept this merge, but thanks for suggesting them.
Take a diff or view commits please.