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.

1 files changed, 3 insertions, 1 deletions

diff --git a/org/madore/damlengine/TodoDamlElement.java b/org/madore/damlengine/TodoDamlElement.java
index 419e7d3..9885f36 100644
--- a/org/madore/damlengine/TodoDamlElement.java
+++ b/org/madore/damlengine/TodoDamlElement.java
@@ -4,9 +4,10 @@ import java.util.ArrayList;
 import java.util.regex.Pattern;
 import org.w3c.dom.*;
 
-public class TodoDamlElement extends TodoDefaultElement {
+public final class TodoDamlElement extends TodoDefaultElement {
 
     public static class Factory extends TodoElement.Factory {
+	@Override
 	public TodoDamlElement newItem(Element node,
 				       Context ctx,
 				       TodoItem caller) {
@@ -20,6 +21,7 @@ public class TodoDamlElement extends TodoDefaultElement {
 	super(node, ctx, caller);
     }
 
+    @Override
     public void handleNodeOnly() {
 	if ( ! ( caller instanceof DamlEngine.RootTodo ) )
 	    throw new IllegalArgumentException("daml node can only be root node");