[Clirr-devel] CVS: clirr/src/java/net/sf/clirr/checks GenderChangeCheck.java,1.10,1.11
Status: Alpha
Brought to you by:
lkuehne
From: Simon K. <s_k...@us...> - 2004-07-02 02:37:36
|
Update of /cvsroot/clirr/clirr/src/java/net/sf/clirr/checks In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv16205 Modified Files: GenderChangeCheck.java Log Message: Use Message class Index: GenderChangeCheck.java =================================================================== RCS file: /cvsroot/clirr/clirr/src/java/net/sf/clirr/checks/GenderChangeCheck.java,v retrieving revision 1.10 retrieving revision 1.11 diff -u -r1.10 -r1.11 --- GenderChangeCheck.java 21 Jun 2004 09:02:33 -0000 1.10 +++ GenderChangeCheck.java 2 Jul 2004 02:37:28 -0000 1.11 @@ -19,8 +19,8 @@ package net.sf.clirr.checks; -import net.sf.clirr.event.ApiDifference; import net.sf.clirr.event.Severity; +import net.sf.clirr.event.Message; import net.sf.clirr.framework.AbstractDiffReporter; import net.sf.clirr.framework.ApiDiffDispatcher; import net.sf.clirr.framework.ClassChangeCheck; @@ -35,6 +35,8 @@ extends AbstractDiffReporter implements ClassChangeCheck { + private static final Message MSG_GENDER_CLASS_TO_INTERFACE = new Message(2000); + private static final Message MSG_GENDER_INTERFACE_TO_CLASS = new Message(2001); /** * Create a new instance of this check. @@ -49,12 +51,15 @@ /** {@inheritDoc} */ public boolean check(JavaClass baseLine, JavaClass current) { - if (baseLine.isClass() != current.isClass()) + if (baseLine.isClass() && current.isInterface()) { - getApiDiffDispatcher().fireDiff(new ApiDifference( - "Changed Gender of " + baseLine.getClassName(), Severity.ERROR, - baseLine.getClassName(), null, null) - ); + log(MSG_GENDER_CLASS_TO_INTERFACE, + Severity.ERROR, baseLine.getClassName(), null, null, null); + } + else if (baseLine.isInterface() && current.isClass()) + { + log(MSG_GENDER_INTERFACE_TO_CLASS, + Severity.ERROR, baseLine.getClassName(), null, null, null); } return true; |