diff options
author | David A. Madore <david+git@madore.org> | 2010-04-22 02:16:29 +0200 |
---|---|---|
committer | David A. Madore <david+git@madore.org> | 2010-04-22 02:16:29 +0200 |
commit | f2745aadc7d0eb02002f667cd72d8536e4f1daf1 (patch) | |
tree | ab03ed4bed1c1cbe9568b3a9f24db15639e41b8d /org/madore/damlengine/TodoXemptyAttr.java | |
parent | 88ef0fa3b0f0ec58cff7d819f9447064be9b6fad (diff) | |
download | damlengine-f2745aadc7d0eb02002f667cd72d8536e4f1daf1.tar.gz damlengine-f2745aadc7d0eb02002f667cd72d8536e4f1daf1.tar.bz2 damlengine-f2745aadc7d0eb02002f667cd72d8536e4f1daf1.zip |
Various "stylistic" improvements suggested by Joshua Bloch's book.
Use @Override annotation. Limit accessibility of fields. Make final
what can be. Use complex enums. Don't ignore exceptions. Some more
changes on exceptions thrown.
Diffstat (limited to 'org/madore/damlengine/TodoXemptyAttr.java')
-rw-r--r-- | org/madore/damlengine/TodoXemptyAttr.java | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/org/madore/damlengine/TodoXemptyAttr.java b/org/madore/damlengine/TodoXemptyAttr.java index cc8491c..a9f7c94 100644 --- a/org/madore/damlengine/TodoXemptyAttr.java +++ b/org/madore/damlengine/TodoXemptyAttr.java @@ -5,6 +5,7 @@ import org.w3c.dom.*; public class TodoXemptyAttr extends TodoAttr { public static class Factory extends TodoAttr.Factory { + @Override public TodoXemptyAttr newItem(Attr attr, Element owner, Context ctx, TodoItem caller) { @@ -18,6 +19,7 @@ public class TodoXemptyAttr extends TodoAttr { super(attr, owner, ctx, caller); } + @Override public void handle() { this.owner.removeAttribute(this.attr.getName()); this.owner.appendChild(ctx.doc.createComment(" EMPTY ")); |