http://codereview.appspot.com/2241042/diff/1/src/com/google/caja/parser/quasiliteral/opt/ArrayIndexOptimization.java
File
src/com/google/caja/parser/quasiliteral/opt/ArrayIndexOptimization.java
(right):

http://codereview.appspot.com/2241042/diff/1/src/com/google/caja/parser/quasiliteral/opt/ArrayIndexOptimization.java#newcode40
src/com/google/caja/parser/quasiliteral/opt/ArrayIndexOptimization.java:40:
* <p>The CajitaRewriter will optimize reads of properties where the
index
Update to reference ES53 (which also does this)

http://codereview.appspot.com/2241042/diff/1/src/com/google/caja/plugin/PipelineMaker.java
File src/com/google/caja/plugin/PipelineMaker.java (right):

http://codereview.appspot.com/2241042/diff/1/src/com/google/caja/plugin/PipelineMaker.java#newcode236
src/com/google/caja/plugin/PipelineMaker.java:236: out.add(new
OptimizeJavascriptStage());
Can we get no-optimize as an option?

http://codereview.appspot.com/2241042/diff/1/tests/com/google/caja/parser/quasiliteral/opt/ArrayIndexOptimizationTest.java
File
tests/com/google/caja/parser/quasiliteral/opt/ArrayIndexOptimizationTest.java
(right):

http://codereview.appspot.com/2241042/diff/1/tests/com/google/caja/parser/quasiliteral/opt/ArrayIndexOptimizationTest.java#newcode138
tests/com/google/caja/parser/quasiliteral/opt/ArrayIndexOptimizationTest.java:138:
+ "function map(f, arr) {\n"
I don't get it - how could this possibly be the optimization of the
reference example?

http://codereview.appspot.com/2241042/

Reply via email to