2009/11/19 <[email protected]>: > LGTM > > > http://codereview.appspot.com/157079/diff/1/12 > File src/com/google/caja/parser/js/Operation.java (right): > > http://codereview.appspot.com/157079/diff/1/12#newcode681 > src/com/google/caja/parser/js/Operation.java:681: //long l = (long) n; > remove old code > > http://codereview.appspot.com/157079/diff/1/2 > File tests/com/google/caja/parser/js/ExpressionTest.java (right): > > http://codereview.appspot.com/157079/diff/1/2#newcode216 > tests/com/google/caja/parser/js/ExpressionTest.java:216: > assertFolded("-2", "-1 ^ 1"); > Do you do any folding of modular arithmetic expressions?
By modular, do you mean using %? > http://codereview.appspot.com/157079 >
