Gijs Kruitbosch wrote:
On 14/05/2015 18:12, Martin Thomson wrote:
I have no idea what you are talking about. I have never had cause to
use new String() anywhere. .toString() maybe.
There are more than 1000 hits for "new String(" in MXR, so our
codebase disagrees.
Most if these are in JS conformance tests or Java source files. However
NotificationDB.jsm should probably be fixed to use
this.runningTask.defer.reject(String(err)); (or err.toString() if it works).
I do remember a case where I switched(!) some code from if (x == <N>) to
use switch() instead and that stopped working because x was actually a
string (from an attribute which was expected to have a numeric value).
--
Warning: May contain traces of nuts.
_______________________________________________
dev-platform mailing list
dev-platform@lists.mozilla.org
https://lists.mozilla.org/listinfo/dev-platform
