http://codereview.appspot.com/89073/diff/10/1007
File tests/com/google/caja/plugin/templates/TemplateCompilerTest.java
(right):

http://codereview.appspot.com/89073/diff/10/1007#newcode302
Line 302: public void test1057ClassNames() throws Exception {
Ah got it now. As in bug number 1057. It's a good idea imho, but we have
not done this anywhere else in our codebase, so it seems jarring in that
context. Perhaps rename the method and add a URL to the bug report in a
comment?

http://codereview.appspot.com/89073

Reply via email to