Skip to content

Commit 210a1a2

Browse files
Calixteromani
authored andcommitted
Issue #169: update to Checkstyle 8.25
1 parent 8a6f981 commit 210a1a2

File tree

11 files changed

+12
-21
lines changed

11 files changed

+12
-21
lines changed

net.sf.eclipsecs.checkstyle/.classpath

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
<?xml version="1.0" encoding="UTF-8"?>
22
<classpath>
3-
<classpathentry exported="true" kind="lib" path="checkstyle-8.24-all.jar" sourcepath="checkstyle-checkstyle-8.24.zip"/>
3+
<classpathentry exported="true" kind="lib" path="checkstyle-8.25-all.jar" sourcepath="checkstyle-checkstyle-8.25.zip"/>
44
<classpathentry kind="con" path="org.eclipse.jdt.launching.JRE_CONTAINER/org.eclipse.jdt.internal.debug.ui.launcher.StandardVMType/JavaSE-1.8">
55
<attributes>
66
<attribute name="maven.pomderived" value="true"/>

net.sf.eclipsecs.checkstyle/META-INF/MANIFEST.MF

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,5 +46,5 @@ Export-Package: .,
4646
com.puppycrawl.tools.checkstyle.utils,
4747
org.apache.commons.beanutils;version="8.24.0"
4848
Bundle-ClassPath: .,
49-
checkstyle-8.24-all.jar
49+
checkstyle-8.25-all.jar
5050
Automatic-Module-Name: net.sf.eclipsecs.checkstyle
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
bin.includes = META-INF/,\
22
.,\
33
license/,\
4-
checkstyle-8.24-all.jar
4+
checkstyle-8.25-all.jar
55
jars.compile.order = .
66
source.. = metadata/

net.sf.eclipsecs.checkstyle/metadata/com/puppycrawl/tools/checkstyle/checks/javadoc/checkstyle-metadata.properties

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,9 @@
11

22
Javadoc.group = Javadoc Comments
33

4-
JavadocMethod.minLineCount = Minimal amount of lines in method to demand documentation presence.
54
JavadocMethod.allowedAnnotations = List of annotations that could allow missed documentation.
65
JavadocMethod.validateThrows = Allows validating throws tags.
7-
JavadocMethod.allowMissingJavadoc = Whether to ignore errors when a method javadoc is missed
86
JavadocMethod.allowMissingParamTags = Whether to ignore errors when a method has parameters but does not have matching param tags in the javadoc.
9-
JavadocMethod.allowMissingPropertyJavadoc = Whether to allow missing Javadoc on accessor methods for properties (setters and getters).
107
JavadocMethod.allowMissingReturnTag = Whether to ignore errors when a method returns non-void type does have a return tag in the javadoc.
118
JavadocMethod.allowMissingThrowsTags = Whether to ignore errors when a method declares that it throws exceptions but does have matching throws tags in the javadoc.
129
JavadocMethod.allowThrowsTagsForSubclasses = Whether to allow documented exceptions that are subclass of one of declared exception.
@@ -18,7 +15,6 @@ JavadocMethod.name = Method Javadoc
1815
JavadocMethod.scope = Visibility scope where Javadoc comments are checked.
1916
JavadocMethod.suppressLoadErrors = Controls whether to show class loading errors in the checkstyle report.
2017
JavadocMethod.tokens = Tokens to check.
21-
JavadocMethod.ignoreMethodNamesRegex = Ignore method whose names are matching specified regex.
2218
2319
JavadocPackage.allowLegacy = If set then allow the use of a package.html file.
2420
JavadocPackage.desc = Checks that each Java package has a Javadoc comment. By default it only allows a <code>package-info.java</code> file, but can be configured to allow a <code>package.html</code> file. \r\n<br/>\r\nAn error will be reported if both files exist as this is not allowed by the Javadoc tool. \r\n

net.sf.eclipsecs.checkstyle/metadata/com/puppycrawl/tools/checkstyle/checks/javadoc/checkstyle-metadata.xml

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -21,9 +21,6 @@
2121
<rule-metadata name="%JavadocMethod.name" internal-name="JavadocMethod" parent="TreeWalker">
2222
<alternative-name internal-name="com.puppycrawl.tools.checkstyle.checks.javadoc.JavadocMethodCheck" />
2323
<description>%JavadocMethod.desc</description>
24-
<property-metadata name="minLineCount" datatype="Integer" default-value="-1">
25-
<description>%JavadocMethod.minLineCount</description>
26-
</property-metadata>
2724
<property-metadata name="allowedAnnotations" datatype="String">
2825
<description>%JavadocMethod.allowedAnnotations</description>
2926
</property-metadata>
@@ -53,12 +50,6 @@
5350
<property-metadata name="allowMissingReturnTag" datatype="Boolean" default-value="false">
5451
<description>%JavadocMethod.allowMissingReturnTag</description>
5552
</property-metadata>
56-
<property-metadata name="allowMissingJavadoc" datatype="Boolean" default-value="false">
57-
<description>%JavadocMethod.allowMissingJavadoc</description>
58-
</property-metadata>
59-
<property-metadata name="allowMissingPropertyJavadoc" datatype="Boolean" default-value="false">
60-
<description>%JavadocMethod.allowMissingPropertyJavadoc</description>
61-
</property-metadata>
6253
<property-metadata name="logLoadErrors" datatype="Boolean" default-value="true">
6354
<description>%JavadocMethod.logLoadErrors</description>
6455
</property-metadata>
@@ -74,9 +65,6 @@
7465
<property-value-option value="ANNOTATION_FIELD_DEF" />
7566
</enumeration>
7667
</property-metadata>
77-
<property-metadata name="ignoreMethodNamesRegex" datatype="Regex">
78-
<description>%JavadocMethod.ignoreMethodNamesRegex</description>
79-
</property-metadata>
8068
<message-key key="javadoc.classInfo" />
8169
<message-key key="javadoc.unusedTagGeneral" />
8270
<message-key key="javadoc.invalidInheritDoc" />

net.sf.eclipsecs.checkstyle/metadata/com/puppycrawl/tools/checkstyle/checks/regexp/checkstyle-metadata.properties

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ RegexpMultiline.maximum = The maximum number of matches required in each file
88
RegexpMultiline.message = message which is used to notify about violations, if empty then default(hard-coded) message is used.
99
RegexpMultiline.minimum = The minimum number of matches required in each file.
1010
RegexpMultiline.name = RegexpMultiline
11+
RegexpMultiline.matchAcrossLines = Controls whether to match expressions across multiple lines.
1112
RegexpMultiline.fileExtensions = file type extension of files to process
1213

1314
RegexpSingleline.desc = A check for detecting single lines that match a supplied regular expression. Works with any file type.\r\n<br/><br/>\r\nRationale: This check can be used to prototype checks and to find common bad practice such as calling ex.printStacktrace(), System.out.println(), System.exit(), etc.

net.sf.eclipsecs.checkstyle/metadata/com/puppycrawl/tools/checkstyle/checks/regexp/checkstyle-metadata.xml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -49,6 +49,9 @@
4949
<property-metadata name="maximum" datatype="Integer" default-value="0">
5050
<description>%RegexpMultiline.maximum</description>
5151
</property-metadata>
52+
<property-metadata name="matchAcrossLines" datatype="Boolean" default-value="false">
53+
<description>%RegexpMultiline.matchAcrossLines</description>
54+
</property-metadata>
5255
<property-metadata name="fileExtensions" datatype="String">
5356
<description>%RegexpMultiline.fileExtensions</description>
5457
</property-metadata>

net.sf.eclipsecs.core/src/net/sf/eclipsecs/core/config/CheckConfigurationTester.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,7 @@
2222

2323
import com.google.common.io.Closeables;
2424
import com.puppycrawl.tools.checkstyle.ConfigurationLoader;
25+
import com.puppycrawl.tools.checkstyle.ConfigurationLoader.IgnoredModulesOptions;
2526
import com.puppycrawl.tools.checkstyle.PropertyResolver;
2627
import com.puppycrawl.tools.checkstyle.api.CheckstyleException;
2728

@@ -96,7 +97,7 @@ public List<ResolvableProperty> getUnresolvedProperties() throws CheckstylePlugi
9697
InputSource in = null;
9798
try {
9899
in = configFile.getCheckConfigFileInputSource();
99-
ConfigurationLoader.loadConfiguration(in, resolver, false);
100+
ConfigurationLoader.loadConfiguration(in, resolver, IgnoredModulesOptions.EXECUTE);
100101
} catch (CheckstyleException e) {
101102
CheckstylePluginException.rethrow(e);
102103
} finally {

0 commit comments

Comments
 (0)