Commits

Doc authored and md_5 committed 11dfcae71a0
#1462: Fix broken links and minor improvement for checkstyle.xml
No tags

checkstyle.xml

Modified
1 1 <?xml version="1.0"?>
2 2 <!DOCTYPE module PUBLIC
3 - "-//Checkstyle//DTD Checkstyle Configuration 1.3//EN"
4 - "https://checkstyle.org/dtds/configuration_1_3.dtd">
3 + "-//Checkstyle//DTD Checkstyle Configuration 1.3//EN"
4 + "https://checkstyle.org/dtds/configuration_1_3.dtd">
5 5
6 6 <module name="Checker">
7 - <!-- See http://checkstyle.sourceforge.net/config_misc.html#NewlineAtEndOfFile -->
7 + <!-- See https://checkstyle.org/checks/misc/newlineatendoffile.html#NewlineAtEndOfFile -->
8 8 <module name="NewlineAtEndOfFile">
9 9 <property name="lineSeparator" value="lf_cr_crlf"/>
10 10 </module>
11 11
12 - <!-- See http://checkstyle.sourceforge.net/config_whitespace.html -->
12 + <!-- See https://checkstyle.org/checks/whitespace/filetabcharacter.html#FileTabCharacter -->
13 13 <module name="FileTabCharacter"/>
14 14
15 - <!-- See http://checkstyle.sourceforge.net/config_misc.html -->
15 + <!-- See https://checkstyle.org/checks/regexp/regexpsingleline.html -->
16 16 <module name="RegexpSingleline">
17 17 <property name="format" value="\s+$"/>
18 18 <property name="minimum" value="0"/>
19 19 <property name="maximum" value="0"/>
20 20 <property name="message" value="Line has trailing spaces."/>
21 21 </module>
22 -
23 - <!-- See http://checkstyle.sourceforge.net/config_filters.html -->
24 - <module name="SuppressionSingleFilter">
25 - <property name="checks" value="UnusedImports"/>
26 - <property name="files" value=".*[/\\]net[/\\]minecraft[/\\].*"/>
27 - </module>
28 -
29 22 <!-- See SPIGOT-7676: Enforce Locale, to prevent issues with turkish 'I' and similar -->
30 23 <module name="RegexpSingleline">
31 - <property name="format" value="\.toUpperCase\(\s*\)" />
32 - <property name="message" value="Use toUpperCase(Locale.ROOT) instead of toUpperCase()" />
24 + <property name="format" value="\.toUpperCase\(\s*\)"/>
25 + <property name="message" value="Use toUpperCase(Locale.ROOT) instead of toUpperCase()"/>
33 26 </module>
34 27 <module name="RegexpSingleline">
35 - <property name="format" value="\.toLowerCase\(\s*\)" />
36 - <property name="message" value="Use toLowerCase(Locale.ROOT) instead of toLowerCase()" />
28 + <property name="format" value="\.toLowerCase\(\s*\)"/>
29 + <property name="message" value="Use toLowerCase(Locale.ROOT) instead of toLowerCase()"/>
30 + </module>
31 +
32 + <!-- See https://checkstyle.org/filters/suppressionsinglefilter.html#SuppressionSingleFilter -->
33 + <module name="SuppressionSingleFilter">
34 + <property name="checks" value="UnusedImports"/>
35 + <property name="files" value=".*[/\\]net[/\\]minecraft[/\\].*"/>
37 36 </module>
38 37
39 38 <module name="TreeWalker">
40 - <!-- See https://checkstyle.org/config_javadoc.html -->
39 + <!-- See https://checkstyle.org/checks/javadoc/index.html -->
41 40 <module name="AtclauseOrder"/>
42 41 <module name="InvalidJavadocPosition"/>
43 42 <module name="JavadocBlockTagLocation"/>
44 43 <module name="JavadocContentLocationCheck"/>
45 44 <!--<module name="JavadocMethod"/>-->
46 45 <module name="JavadocType"/>
47 46 <module name="MissingJavadocPackage"/>
48 47 <!--<module name="NonEmptyAtclauseDescription"/>-->
49 48
50 - <!-- See http://checkstyle.sourceforge.net/config_filters.html -->
49 + <!-- See https://checkstyle.org/filters/index.html -->
51 50 <module name="SuppressionCommentFilter"/>
52 51 <module name="SuppressionCommentFilter">
53 52 <property name="offCommentFormat" value="CraftBukkit start"/>
54 53 <property name="onCommentFormat" value="CraftBukkit end"/>
55 54 </module>
56 55 <module name="SuppressWithNearbyCommentFilter">
57 56 <property name="commentFormat" value="CraftBukkit"/>
58 57 </module>
59 58
60 - <!-- See http://checkstyle.sourceforge.net/config_imports.html -->
59 + <!-- See https://checkstyle.org/checks/imports/index.html -->
61 60 <module name="AvoidStarImport">
62 61 <property name="allowStaticMemberImports" value="true"/>
63 62 </module>
64 63 <module name="ImportOrder">
65 64 <property name="option" value="above"/>
66 65 <property name="ordered" value="true"/>
67 66 <property name="separatedStaticGroups" value="true"/>
68 67 <property name="sortStaticImportsAlphabetically" value="true"/>
69 68 </module>
70 69 <module name="RedundantImport"/>
71 70 <module name="UnusedImports"/>
72 71
73 - <!-- See https://checkstyle.org/config_whitespace.html -->
72 + <!-- See https://checkstyle.org/checks/whitespace/index.html -->
74 73 <module name="GenericWhitespace"/>
75 74 <module name="MethodParamPad"/>
76 75 <module name="NoLineWrap"/>
77 76 <module name="NoWhitespaceAfter"/>
78 77 <module name="NoWhitespaceBefore">
79 78 <property name="allowLineBreaks" value="true"/>
80 79 </module>
81 80 <module name="OperatorWrap"/>
82 81 <module name="ParenPad"/>
83 82 <module name="SingleSpaceSeparator"/>
84 83 <module name="TypecastParenPad"/>
85 84 <module name="WhitespaceAfter"/>
86 85 <module name="WhitespaceAround">
87 86 <property name="allowEmptyCatches" value="true"/>
88 87 <property name="allowEmptyConstructors" value="true"/>
89 88 <property name="allowEmptyMethods" value="true"/>
90 89 <property name="allowEmptyTypes" value="true"/>
91 90 </module>
92 91
93 - <!-- See http://checkstyle.sourceforge.net/config_modifiers.html -->
92 + <!-- See https://checkstyle.org/checks/modifier/index.html -->
94 93 <module name="ModifierOrder"/>
95 94
96 - <!-- See https://checkstyle.org/config_blocks.html -->
95 + <!-- See https://checkstyle.org/checks/blocks/index.html -->
97 96 <!--<module name="AvoidNestedBlocks"/>-->
98 97 <module name="LeftCurly"/>
99 98 <module name="RightCurly"/>
100 99
101 - <!-- See http://checkstyle.sourceforge.net/config_design.html -->
100 + <!-- See https://checkstyle.org/checks/design/index.html -->
102 101 <!--<module name="FinalClass"/>-->
103 102 <!--<module name="InterfaceIsType"/>-->
104 103
105 - <!-- See http://checkstyle.sourceforge.net/config_misc.html -->
104 + <!-- See https://checkstyle.org/checks/misc/index.html -->
106 105 <module name="ArrayTypeStyle"/>
107 106 <module name="UpperEll"/>
108 107 </module>
109 108 </module>

Everything looks good. We'll let you know here if there's anything you should know about.

Add shortcut