RE: [argouml-dev] The new version of checkclipse

[Linus Tolke]

Hello Michiel!   When doing this kind of changes the most important thing is to stop the adding of new code in violation. Cleaning up the old code needs to be done too but that is a lot of work and can be done while doing other style changes.   If you, or anyone, find this worthwhile, please go ahead. Otherwise we will keep it in the “fix this eventually”-pile.               /Linus   From: Michiel van der Wulp [mailto:[EMAIL PROTECTED] Sent: den 18 oktober 2006 22:40 To: dev@argouml.tigris.org Subject: Re: [argouml-dev] The new version of checkclipse   Ah, OK, maybe I was a bit confused about the "1st problem", i.e. the appearance of JavadocMethod warnings for "@see" style of Javadoc.   This is in fact solved by replacing all the   /*  * @see ... */   by   /*  * [EMAIL PROTECTED]} */   Did we decide to perform this change everywhere?   Linus' mail from yesterday says that we have to use the @inheritDoc from now on, but I presume we also have to eradicate all @see javadocs that give the warning.   Can you clarify Linus?   Regards, Michiel       ----- Original Message ----- From: Michiel van der Wulp To: dev@argouml.tigris.org Sent: Wednesday, October 18, 2006 10:18 PM Subject: Re: [argouml-dev] The new version of checkclipse   Hi Linus, et al.,   I see the same problem. Apparently, this is the second problem with the JavadocMethod check. See also: http://argouml.tigris.org/servlets/ReadMsg?list=dev&msgNo=19813   Luckily, I found a solution today (for this 2nd problem):   Index: C:/Documents and Settings/Michiel/My Documents/Java_Development/ArgoUML_SVN_Workspace/argouml-core-tools/checkstyle/checkstyle_argouml.xml =================================================================== --- C:/Documents and Settings/Michiel/My Documents/Java_Development/ArgoUML_SVN_Workspace/argouml-core-tools/checkstyle/checkstyle_argouml.xml (revision 11286) +++ C:/Documents and Settings/Michiel/My Documents/Java_Development/ArgoUML_SVN_Workspace/argouml-core-tools/checkstyle/checkstyle_argouml.xml (working copy) @@ -77,6 +77,8 @@      <module name="JavadocMethod">        <property name="scope" value="protected"/>        <property name="allowUndeclaredRTE" value="true"/> +      <property name="logLoadErrors" value="true"/> +      <property name="suppressLoadErrors" value="true"/>      </module>      <module name="JavadocVariable">        <property name="scope" value="protected"/> I do not know if both lines are required, but it solves the problem like this. If this works for you too, please commit!   PS: But I fear I will have to comment out the complete JavadocMethod check again, since the 1st problem is still not solved yet.   Regards, Michiel     ----- Original Message ----- From: Linus Tolke To: dev@argouml.tigris.org Sent: Tuesday, October 17, 2006 4:05 PM Subject: RE: [argouml-dev] The new version of checkclipse   I am referring to the Got an exception - java.lang.RuntimeException: Unable to get class information for @throws tag *whatever*. errors.               /Linus   From: Linus Tolke [mailto:[EMAIL PROTECTED] Sent: den 17 oktober 2006 16:03 To: dev@argouml.tigris.org Subject: [argouml-dev] The new version of checkclipse   Hello all!   I attempted to enable checklipse 2.1.0 (the latest available version) in my Eclipse 3.2 installation that previously didn’t have a checklipse.   I noticed that it doesn’t work i.e. it gives a lot of false errors. This seems to be a known problem that has been bounced back and forth a little between the checkstyle and the checklipse project but appearantly not resolved.   What can we do about it? Shall we just wait and accept that we have these false errors? Anything else?               /Linus   ************************************************ Linus Tolke [EMAIL PROTECTED]   [EMAIL PROTECTED] ArgoUML ************************************************   No virus found in this incoming message. Checked by AVG Free Edition. Version: 7.1.408 / Virus Database: 268.13.4/478 - Release Date: 17/10/2006 --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] No virus found in this incoming message. Checked by AVG Free Edition. Version: 7.1.408 / Virus Database: 268.13.5/483 - Release Date: 18/10/2006