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, 6 insertions, 6 deletions

diff --git a/org/madore/damlengine/TodoDeque.java b/org/madore/damlengine/TodoDeque.java
index 4cb1d38..3970938 100644
--- a/org/madore/damlengine/TodoDeque.java
+++ b/org/madore/damlengine/TodoDeque.java
@@ -5,31 +5,31 @@ import java.util.LinkedList;
 
 public final class TodoDeque {
 
-    private LinkedList<TodoItem> deque;
+    protected final LinkedList<TodoItem> deque;
 
     public TodoDeque() {
 	deque = new LinkedList<TodoItem>();
     }
 
     public void registerAtEnd(TodoItem it) {
-	it.ownerDeque = this;
+	it.setOwnerDeque(this);
 	deque.addLast(it);
     }
 
     public void registerAtEnd(Collection<? extends TodoItem> them) {
 	for ( TodoItem it : them )
-	    it.ownerDeque = this;
+	    it.setOwnerDeque(this);
 	deque.addAll(them);
     }
 
     public void registerAtStart(TodoItem it) {
-	it.ownerDeque = this;
+	it.setOwnerDeque(this);
 	deque.addFirst(it);
     }
 
     public void registerAtStart(Collection<? extends TodoItem> them) {
 	for ( TodoItem it : them )
-	    it.ownerDeque = this;
+	    it.setOwnerDeque(this);
 	deque.addAll(0, them);
     }
 
@@ -40,7 +40,7 @@ public final class TodoDeque {
     public boolean dispatchOne() {
 	TodoItem it = removeNext();
 	if ( it != null ) {
-	    assert(it.ownerDeque == this);
+	    assert(it.getOwnerDeque() == this);
 	    it.handle();
 	    return true;
 	} else