svn commit: r15988 - trunk/src/argouml-app/src/org/argouml/cognitive/ToDoList.java

Author: bobtarling
Date: 2008-11-08 08:20:51-0800
New Revision: 15988

Modified:
   trunk/src/argouml-app/src/org/argouml/cognitive/ToDoList.java

Log:
Deprecate all protected methods that should have been private

Modified: trunk/src/argouml-app/src/org/argouml/cognitive/ToDoList.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src/argouml-app/src/org/argouml/cognitive/ToDoList.java?view=diff&rev=15988&p1=trunk/src/argouml-app/src/org/argouml/cognitive/ToDoList.java&p2=trunk/src/argouml-app/src/org/argouml/cognitive/ToDoList.java&r1=15987&r2=15988
==============================================================================
--- trunk/src/argouml-app/src/org/argouml/cognitive/ToDoList.java	(original)
+++ trunk/src/argouml-app/src/org/argouml/cognitive/ToDoList.java	2008-11-08 08:20:51-0800
@@ -210,7 +210,11 @@
      * synchronized the Designer, otherwise there will be deadlock.
      * 
      * @param removes a list containing the items to be removed
+     * @deprecated in 0.27.2 by Bob Tarling. ToDoList is not designed to be
+     * extended so protected is not required. This method is not used with the
+     * package so scope will become private in future.
      */
+    @Deprecated
     protected synchronized void forceValidityCheck(
             final List<ToDoItem> removes) {
         synchronized (items) {
@@ -676,14 +680,22 @@
 
     /**
      * Re-compute all offenders.
+     * @deprecated in 0.27.2 by Bob Tarling. ToDoList is not designed to be
+     * extended so protected is not required. This method is not used with the
+     * package so scope will become private in future.
      */
+    @Deprecated
     protected void recomputeAllOffenders() {
         allOffenders = null;
     }
 
     /**
      * Reset all posters.
+     * @deprecated in 0.27.2 by Bob Tarling. ToDoList is not designed to be
+     * extended so protected is not required. This method is not used with the
+     * package so scope will become private in future.
      */
+    @Deprecated
     protected void recomputeAllPosters() {
         allPosters = null;
     }
@@ -715,7 +727,11 @@
      * parameters passed into the fire method.
      * 
      * @see EventListenerList
+     * @deprecated in 0.27.2 by Bob Tarling. ToDoList is not designed to be
+     * extended so protected is not required. This method is not used with the
+     * package so scope will become private in future.
      */
+    @Deprecated
     protected void fireToDoListChanged() {
         // Guaranteed to return a non-null array
         Object[] listeners = listenerList.getListenerList();
@@ -735,7 +751,11 @@
 
     /**
      * @param item the todo item
+     * @deprecated in 0.27.2 by Bob Tarling. ToDoList is not designed to be
+     * extended so protected is not required. This method is not used with the
+     * package so scope will become private in future.
      */
+    @Deprecated
     protected void fireToDoItemChanged(ToDoItem item) {
         Object[] listeners = listenerList.getListenerList();
         ToDoListEvent e = null;
@@ -756,6 +776,10 @@
 
     /**
      * @param item the todo item
+     * @deprecated in 0.27.2 by Bob Tarling. ToDoList is not designed to be
+     * extended so protected is not required. This method is not used with the
+     * package so scope will become private in future.
+     * @Deprecated
      */
     protected void fireToDoItemAdded(ToDoItem item) {
         List<ToDoItem> l = new ArrayList<ToDoItem>();
@@ -765,7 +789,11 @@
 
     /**
      * @param theItems the todo items
+     * @deprecated in 0.27.2 by Bob Tarling. ToDoList is not designed to be
+     * extended so protected is not required. This method is not used with the
+     * package so scope will become private in future.
      */
+    @Deprecated
     protected void fireToDoItemsAdded(List<ToDoItem> theItems) {
         if (theItems.size() > 0) {
             // Guaranteed to return a non-null array
@@ -787,7 +815,11 @@
 
     /**
      * @param item the todo item
+     * @deprecated in 0.27.2 by Bob Tarling. ToDoList is not designed to be
+     * extended so protected is not required. This method is not used with the
+     * package so scope will become private in future.
      */
+    @Deprecated
     protected void fireToDoItemRemoved(ToDoItem item) {
         List<ToDoItem> l = new ArrayList<ToDoItem>();
         l.add(item);
@@ -796,7 +828,11 @@
 
     /**
      * @param theItems the todo items
+     * @deprecated in 0.27.2 by Bob Tarling. ToDoList is not designed to be
+     * extended so protected is not required. This method is not used with the
+     * package so scope will become private in future.
      */
+    @Deprecated
     protected void fireToDoItemsRemoved(final List<ToDoItem> theItems) {
         if (theItems.size() > 0) {
             // Guaranteed to return a non-null array