Much agreed with Jonny. That code has ugly separators at the end of lines and jumped out immediately.
Besides, emacs would kindly have underlined that bit in red for me. (and turned yellow any globals) On Feb 23, 2:48 pm, Jonathan Buchanan <[email protected]> wrote: > On 23 February 2012 14:29, Dean Landolt <[email protected]> wrote: > > > > > On Thu, Feb 23, 2012 at 4:44 AM, Arnout Kazemier <[email protected]> wrote: > > >> Comma first solves that perfectly, no need to jshint that ;D > > > I wouldn't say perfectly. There's a gaping hole in the comma-first style: > > > var arr = [ "foo", > > , "bar", > > , "baz" > > ] > > > Did you spot the error right away? > > Yes - this code screamed "I AM WROOONNNNG!" at me before I even read > the sentence above it. > > I'm used to writing code with comma-first and only using semicolons > where required, so punctuation at the end of a line stands out an > absolute mile. > > -- > Jonny -- Job Board: http://jobs.nodejs.org/ Posting guidelines: https://github.com/joyent/node/wiki/Mailing-List-Posting-Guidelines You received this message because you are subscribed to the Google Groups "nodejs" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [email protected] For more options, visit this group at http://groups.google.com/group/nodejs?hl=en?hl=en
