On 03/28/2014 02:01 PM, Andrew Sutherland wrote:
> It's my understanding that the existing idiom used throughout the Thunderbird
> tree is still okay to do since the prototype chain is created at object
> initialization time and so there's no actual mutation of the chain:
>
> function Application() {
> }
> Application.prototype = {
> __proto__: extApplication.prototype,
> };
At present we only warn for direct property-sets not part of object literals.
This was because property-sets are more visibly, obviously, clearly wrong in
this regard, while *in theory* the other form could be made not so bad.
But right now, that pattern above is just as bad in SpiderMonkey. And,
honestly, I suspect we'll have other performance work and features to implement
for a long time to come. So really you shouldn't do this pattern, either. But
we don't warn about it, just yet, to minimize warning fatigue.
The right fix -- judging by the spew/code earlier in this thread -- is probably
to port the patch from <https://bugzilla.mozilla.org/show_bug.cgi?id=985742>.
Jeff
_______________________________________________
dev-platform mailing list
dev-platform@lists.mozilla.org
https://lists.mozilla.org/listinfo/dev-platform
