http://codereview.appspot.com/110079/diff/1/7
File tests/com/google/caja/opensocial/example-rewritten.xml (right):

http://codereview.appspot.com/110079/diff/1/7#newcode31
Line 31: ;
Why is this semi on the next line?

http://codereview.appspot.com/110079/diff/1/2
File tests/com/google/caja/render/ssp-golden.txt (right):

http://codereview.appspot.com/110079/diff/1/2#newcode7
Line 7: [0,,,]
I'm not competent to judge whether this is correct.
Ihab, can you take a look at this.

http://codereview.appspot.com/110079/diff/1/9
File tests/com/google/caja/service/CajolingServiceTest.java (right):

http://codereview.appspot.com/110079/diff/1/9#newcode219
Line 219: String indent = "    ";
I think this is defunct now, since the spaces take less space than the +
indent + " below.

http://codereview.appspot.com/110079

Reply via email to