http://codereview.appspot.com/4610043/diff/7/src/com/google/caja/reporting/RenderContext.java
File src/com/google/caja/reporting/RenderContext.java (right):

http://codereview.appspot.com/4610043/diff/7/src/com/google/caja/reporting/RenderContext.java#newcode86
src/com/google/caja/reporting/RenderContext.java:86: // Enforce that
derive has been overridden.
On 2011/06/14 22:47:34, MikeSamuel wrote:
yep.  Not when we run our tests though, so we will detect any tested
classes
that do this.  I could always change this to throw new AssertionError
if you
think it is worthwhile to have this always on.

catching in testing is probably enough.

http://codereview.appspot.com/4610043/

Reply via email to