Changes between Initial Version and Version 1 of Ticket #19592, comment 26
- Timestamp:
- 03/03/2012 09:08:23 PM (14 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #19592, comment 26
initial v1 1 1 Replying to [comment:25 soulseekah]: 2 2 > Most of the warnings are "try to use only one {{{ --> var <-- }}}". Readability of JS code is severely impaired when fixing these... 3 > 3 > ... 4 4 > Most of these are best left alone, not really worth chasing down the 3 extra bytes here and there... 5 5 6 This is rather a question of coding standards not of trying to save few bytes. It is a good practice to declare all local variables at the beginning of each scope (i.e. have one `var ...;` at the top of each function). This actually helps readability and is preferred in most (all?) IDEs that have JS support. (It also seems to simplify the actualminifying in YUI Compressor).6 This is rather a question of coding standards not of trying to save few bytes. It is a good practice to declare all local variables at the beginning of each scope (i.e. have one `var ...;` at the top of each function). This actually helps readability and is preferred in most (all?) IDEs that have JS support. (It also seems to simplify the minifying in YUI Compressor). 7 7 8 8 > Overall should we be listening to some sort of Lint instead of a compressor...