[Clirr-devel] CVS: clirr/core/src/java/net/sf/clirr/core/internal/checks ClassScopeCheck.java,1.2,1.3

[Simon K. <s_k...@us...>]

Update of /cvsroot/clirr/clirr/core/src/java/net/sf/clirr/core/internal/checks
In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv7503

Modified Files:
	ClassScopeCheck.java 
Log Message:
Use new getSeverity method to suppress ERRORs for package & private diffs.
Also wrapped some overly-long lines


Index: ClassScopeCheck.java
===================================================================
RCS file: /cvsroot/clirr/clirr/core/src/java/net/sf/clirr/core/internal/checks/ClassScopeCheck.java,v
retrieving revision 1.2
retrieving revision 1.3
diff -u -r1.2 -r1.3
--- ClassScopeCheck.java	11 Jul 2004 10:33:50 -0000	1.2
+++ ClassScopeCheck.java	16 Jul 2004 09:47:24 -0000	1.3
@@ -46,7 +46,9 @@
  *
  * @author Simon Kitching
  */
-public final class ClassScopeCheck extends AbstractDiffReporter implements ClassChangeCheck
+public final class ClassScopeCheck
+        extends AbstractDiffReporter
+        implements ClassChangeCheck
 {
     private static final Message MSG_SCOPE_INCREASED = new Message(1000);
     private static final Message MSG_SCOPE_DECREASED = new Message(1001);
@@ -75,7 +77,9 @@
         }
         catch (CheckerException ex)
         {
-            log(MSG_ERROR_DETERMINING_SCOPE_OLD, Severity.ERROR, compatBaseline.getClassName(), null, null, new String[]{ex.getMessage()});
+            log(MSG_ERROR_DETERMINING_SCOPE_OLD,
+                Severity.ERROR, compatBaseline.getClassName(), null, null,
+                new String[] {ex.getMessage()});
             return false;
         }
 
@@ -86,7 +90,9 @@
         }
         catch (CheckerException ex)
         {
-            log(MSG_ERROR_DETERMINING_SCOPE_NEW, Severity.ERROR, compatBaseline.getClassName(), null, null, new String[]{ex.getMessage()});
+            log(MSG_ERROR_DETERMINING_SCOPE_NEW,
+                Severity.ERROR, compatBaseline.getClassName(), null, null,
+                new String[] {ex.getMessage()});
             return false;
         }
 
@@ -102,13 +108,16 @@
         {
             String[] args = {bScope.getDesc(), cScope.getDesc()};
 
-            log(MSG_SCOPE_INCREASED, Severity.INFO, compatBaseline.getClassName(), null, null, args);
+            log(MSG_SCOPE_INCREASED,
+                Severity.INFO, compatBaseline.getClassName(), null, null, args);
         }
         else if (cScope.isLessVisibleThan(bScope))
         {
             String[] args = {bScope.getDesc(), cScope.getDesc()};
 
-            log(MSG_SCOPE_DECREASED, Severity.ERROR, compatBaseline.getClassName(), null, null, args);
+            log(MSG_SCOPE_DECREASED,
+                getSeverity(compatBaseline, Severity.ERROR),
+                compatBaseline.getClassName(), null, null, args);
         }
 
         // Apply further checks only if both versions of the class have scopes