From 5fe300ec4252541d457ae5a3d01ac223480959b4 Mon Sep 17 00:00:00 2001
From: Doc <nachito94@msn.com>
Date: Mon, 12 Aug 2024 21:26:44 +1000
Subject: [PATCH] #1052: Fix broken links and minor improvement for
 checkstyle.xml

---
 checkstyle.xml | 48 ++++++++++++++++++++++++------------------------
 1 file changed, 24 insertions(+), 24 deletions(-)

diff --git a/checkstyle.xml b/checkstyle.xml
index 118f0e3f..1d1e06c8 100644
--- a/checkstyle.xml
+++ b/checkstyle.xml
@@ -1,28 +1,38 @@
 <?xml version="1.0"?>
 <!DOCTYPE module PUBLIC
-          "-//Checkstyle//DTD Checkstyle Configuration 1.3//EN"
-          "https://checkstyle.org/dtds/configuration_1_3.dtd">
+        "-//Checkstyle//DTD Checkstyle Configuration 1.3//EN"
+        "https://checkstyle.org/dtds/configuration_1_3.dtd">
 
 <module name="Checker">
-    <!-- See http://checkstyle.sourceforge.net/config_misc.html#NewlineAtEndOfFile -->
+    <!-- See https://checkstyle.org/checks/misc/newlineatendoffile.html#NewlineAtEndOfFile -->
     <module name="NewlineAtEndOfFile">
         <property name="lineSeparator" value="lf_cr_crlf"/>
     </module>
 
-    <!-- See http://checkstyle.sourceforge.net/config_whitespace.html -->
+    <!-- See https://checkstyle.org/checks/whitespace/filetabcharacter.html#FileTabCharacter -->
     <module name="FileTabCharacter"/>
 
-    <!-- See http://checkstyle.sourceforge.net/config_misc.html -->
+    <!-- See https://checkstyle.org/checks/regexp/regexpsingleline.html -->
     <module name="RegexpSingleline">
         <property name="format" value="\s+$"/>
         <property name="minimum" value="0"/>
         <property name="maximum" value="0"/>
         <property name="message" value="Line has trailing spaces."/>
     </module>
+    <!-- See SPIGOT-7676: Enforce Locale, to prevent issues with turkish 'I' and similar -->
+    <module name="RegexpSingleline">
+        <property name="format" value="\.toUpperCase\(\s*\)"/>
+        <property name="message" value="Use toUpperCase(Locale.ROOT) instead of toUpperCase()"/>
+    </module>
+    <module name="RegexpSingleline">
+        <property name="format" value="\.toLowerCase\(\s*\)"/>
+        <property name="message" value="Use toLowerCase(Locale.ROOT) instead of toLowerCase()"/>
+    </module>
 
-    <!-- See https://checkstyle.sourceforge.io/config_javadoc.html#JavadocPackage-->
+    <!-- See https://checkstyle.org/checks/javadoc/javadocpackage.html#JavadocPackage -->
     <module name="JavadocPackage"/>
 
+    <!-- See https://checkstyle.org/filters/suppressionsinglefilter.html#SuppressionSingleFilter -->
     <module name="SuppressionSingleFilter">
         <property name="message" value="'(implNote|implSpec|apiNote)'\."/>
     </module>
@@ -32,18 +42,8 @@
         <property name="files" value=".*[\\/]src[\\/]test[\\/].*\.java$"/>
     </module>
 
-    <!-- See SPIGOT-7676: Enforce Locale, to prevent issues with turkish 'I' and similar -->
-    <module name="RegexpSingleline">
-        <property name="format" value="\.toUpperCase\(\s*\)" />
-        <property name="message" value="Use toUpperCase(Locale.ROOT) instead of toUpperCase()" />
-    </module>
-    <module name="RegexpSingleline">
-        <property name="format" value="\.toLowerCase\(\s*\)" />
-        <property name="message" value="Use toLowerCase(Locale.ROOT) instead of toLowerCase()" />
-    </module>
-
     <module name="TreeWalker">
-        <!-- See https://checkstyle.org/config_javadoc.html -->
+        <!-- See https://checkstyle.org/checks/javadoc/index.html -->
         <module name="AtclauseOrder"/>
         <module name="InvalidJavadocPosition"/>
         <module name="JavadocBlockTagLocation"/>
@@ -53,10 +53,10 @@
         <module name="MissingJavadocPackage"/>
         <module name="NonEmptyAtclauseDescription"/>
 
-        <!-- See http://checkstyle.sourceforge.net/config_filters.html -->
+        <!-- See https://checkstyle.org/filters/index.html -->
         <module name="SuppressionCommentFilter"/>
 
-        <!-- See http://checkstyle.sourceforge.net/config_imports.html -->
+        <!-- See https://checkstyle.org/checks/imports/index.html -->
         <module name="AvoidStarImport">
             <property name="allowStaticMemberImports" value="true"/>
         </module>
@@ -69,7 +69,7 @@
         <module name="RedundantImport"/>
         <module name="UnusedImports"/>
 
-        <!-- See https://checkstyle.org/config_whitespace.html -->
+        <!-- See https://checkstyle.org/checks/whitespace/index.html -->
         <module name="GenericWhitespace"/>
         <module name="MethodParamPad"/>
         <module name="NoLineWrap"/>
@@ -89,18 +89,18 @@
             <property name="allowEmptyTypes" value="true"/>
         </module>
 
-        <!-- See http://checkstyle.sourceforge.net/config_modifiers.html -->
+        <!-- See https://checkstyle.org/checks/modifier/index.html -->
         <module name="ModifierOrder"/>
 
-        <!-- See https://checkstyle.org/config_blocks.html -->
+        <!-- See https://checkstyle.org/checks/blocks/index.html -->
         <module name="AvoidNestedBlocks"/>
         <module name="LeftCurly"/>
         <module name="RightCurly"/>
 
-        <!-- See http://checkstyle.sourceforge.net/config_design.html -->
+        <!-- See https://checkstyle.org/checks/design/index.html -->
         <module name="FinalClass"/>
 
-        <!-- See http://checkstyle.sourceforge.net/config_misc.html -->
+        <!-- See https://checkstyle.org/checks/misc/index.html -->
         <module name="ArrayTypeStyle"/>
         <module name="UpperEll"/>
     </module>