search

hongee / google-caja

Automatically exported from code.google.com/p/google-caja
0 stars 0 forks source link

Review ihab/proto #238

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
Makes __proto__ property accessible.

Original issue reported on code.google.com by ihab.a...@gmail.com on 1 May 2008 at 8:24

GoogleCodeExporter commented 9 years ago

Original comment by metaw...@gmail.com on 2 May 2008 at 11:01

GoogleCodeExporter commented 9 years ago

Original comment by erights on 2 May 2008 at 11:20

GoogleCodeExporter commented 9 years ago 
David-Sarah Hopwood is correct. This is a misconceived and ill-motivated CL 
which is
hereby rejected. Everywhere in Prototype.js (or any other library) where this CL
would have applied, i.e., where there was an expression of the form

  <expr>.__proto__

It should be replaced with

  void <expr>

In the typical case where the <expr> itself has no side effects, it should 
simply be
replaced with

  void 0

Original comment by erights on 3 May 2008 at 12:41