I was the person in the previous thread who found them useful, and I still
do. Some of the extraWarnings stuff is of questionable value, but the
undefined property stuff is really useful.

I don't really know if other people use this stuff. extraWarnings are
enabled by default in debug builds, and I think a lot of people don't
realize the difference between these "extra warnings" and normal JS errors.

Writing front-end JS code for Firefox is generally a painful experience:
terrible error messages, often no filename or line number information, or
even no output at all. But I think we should be making things better, not
worse. Remove features like this takes us in the wrong direction. If the
undefined property warning is broken, we should fix it.

-Bill

On Fri, Dec 19, 2014 at 12:32 PM, J. Ryan Stinnett <jry...@gmail.com> wrote:
>
> Some prior discussion of this feature happened in the platform thread
> "Disabling
> strict warnings as errors in xpcshell"[1].
>
> A few people argued for the extra warnings to be removed, while one person
> said they were useful.
>
> No clear conclusion was reached.
>
> [1]:
>
> https://groups.google.com/d/topic/mozilla.dev.platform/znIkVsh5YYA/discussion
>
> On Fri, Dec 19, 2014 at 2:19 PM, Jason Orendorff <jorendo...@mozilla.com>
> wrote:
> >
> > So if you go to about:config and set the javascript.options.strict pref,
> > you'll get warnings about accessing undefined properties.
> >
> >     js> Math.TAU
> >     undefined
> >     /!\ ReferenceError: reference to undefined property Math.TAU
> >
> > (It says "ReferenceError", but your code still runs normally; it really
> is
> > just a warning.)
> >
> > Is anyone using this? Bug 1113380 points out that the rules about what
> kind
> > of code can cause a warning are a little weird (on purpose, I think).
> Maybe
> > it's time to retire this feature.
> >
> >     https://bugzilla.mozilla.org/show_bug.cgi?id=1113380
> >
> > Please speak up now, if you're still using it!
> >
> > -j
> > _______________________________________________
> > dev-platform mailing list
> > dev-platform@lists.mozilla.org
> > https://lists.mozilla.org/listinfo/dev-platform
> >
> _______________________________________________
> dev-platform mailing list
> dev-platform@lists.mozilla.org
> https://lists.mozilla.org/listinfo/dev-platform
>
_______________________________________________
dev-platform mailing list
dev-platform@lists.mozilla.org
https://lists.mozilla.org/listinfo/dev-platform

Reply via email to