On Tuesday, November 29, 2011 10:51:52 AM UTC+1, Sebo wrote:
>
> Why?
>>
> Just optically I don't like it. You also don't add a comma to the end of 
> other arrays, do you?
>
> [1, 2, 3, ] looks like there is something missing.
>

I would say it's more about following cases.
define([
    "firebug/lib/object",
    "firebug/firebug",
    "firebug/lib/events",
],
function(Obj, Firebug, Events) {
 

> We're discussing a little thing here for quite a while. So should it be 
> changed or not?
>

I personally don't see any real reason why we should adopt such rule (also 
if we wanted to adopt it, we need a mechanism how to detect such places and 
strict-warning is currently not the right way).

This can be more important when we want to share some code with tools that 
run in the browser (cross-browser), but it's not the priority now either.

Honza

 

-- 
You received this message because you are subscribed to the Google
Groups "Firebug" 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
https://groups.google.com/forum/#!forum/firebug

Reply via email to