Good point. I've honestly run into that a few times myself in that file, so if it's not just me misreading it I should definitely change it!

Matthew Riley MacPherson (Sent from mobile)

> On Nov 6, 2013, at 9:34, "sole" <notifications@github.com> wrote:
> 
> In www/js/globals.coffee:
> 
> >  
> >      try
> >        request.send();
> >      catch error
> >        console.log(error)
> >        mockL10n()
> > +
> > +    d.promise()
> >    window.setLanguage = setLanguage
> Minor nitpick, I'd add a \n before this line. Since the setLanguage method is so long and CS doesn't have closing } I wasn't sure whether window.setLanguage = setLanguage belonged to the setLanguage method (evidently not when you think it twice, but it is a tad confusing)
> 
> —
> Reply to this email directly or view it on GitHub.