svn commit: r13549 - trunk/src_new/org/argouml/uml/cognitive/critics/CrEmptyPackage.java

Author: tfmorris
Date: 2007-09-17 12:39:52-0700
New Revision: 13549

Modified:
   trunk/src_new/org/argouml/uml/cognitive/critics/CrEmptyPackage.java

Log:
Make logging message more meaningful.

Modified: trunk/src_new/org/argouml/uml/cognitive/critics/CrEmptyPackage.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src_new/org/argouml/uml/cognitive/critics/CrEmptyPackage.java?view=diff&rev=13549&p1=trunk/src_new/org/argouml/uml/cognitive/critics/CrEmptyPackage.java&p2=trunk/src_new/org/argouml/uml/cognitive/critics/CrEmptyPackage.java&r1=13548&r2=13549
==============================================================================
--- trunk/src_new/org/argouml/uml/cognitive/critics/CrEmptyPackage.java	(original)
+++ trunk/src_new/org/argouml/uml/cognitive/critics/CrEmptyPackage.java	2007-09-17 12:39:52-0700
@@ -33,7 +33,7 @@
 
 
 /**
- * A critic whether a package/subsystem/model is empty.
+ * A critic to check for empty packages.
  *
  * @author Jason Robbins
  */
@@ -49,7 +49,6 @@
 
     /**
      * The constructor.
-     *
      */
     public CrEmptyPackage() {
         setupHeadAndDesc();
@@ -57,22 +56,18 @@
 	addTrigger("ownedElement");
     }
 
-    /*
-     * @see org.argouml.uml.cognitive.critics.CrUML#predicate2(
-     *      java.lang.Object, org.argouml.cognitive.Designer)
-     */
+    @Override
     public boolean predicate2(Object dm, Designer dsgr) {
-//	LOG.debug("predicate2 on " + dm);
 	if (!(Model.getFacade().isAPackage(dm))) {
 	    return NO_PROBLEM;
 	}
 	Collection elems = Model.getFacade().getOwnedElements(dm);
 	if (elems.size() == 0) {
-            LOG.debug("PROBLEM_FOUND on " + dm);
+            LOG.debug("Found empty package " + Model.getFacade().toString(dm));
             return PROBLEM_FOUND;
         }
 	return NO_PROBLEM;
     }
 
-} /* end class CrEmptyPackage */
+}