Raincloud/sheepit-shadow-nabber
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge branch 'pmd/new-rule5' into 'master'

Browse Source 
Ref: add new static analyser rule: category/java/errorprone.xml/UnnecessaryCaseChange

See merge request sheepitrenderfarm/client!318
This commit is contained in:
Laurent Clouet
2024-06-06 01:42:04 +00:00
parent 86c21a96d2 21ab1d0e19
commit 556ab75670
2 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
rulesets/java-sheepit.xml
View File

@@ -49,6 +49,8 @@
<rule ref="category/java/errorprone.xml/UnnecessaryConversionTemporary" /> <rule ref="category/java/errorprone.xml/UnnecessaryConversionTemporary" />
<rule ref="category/java/errorprone.xml/UnusedNullCheckInEquals" /> <rule ref="category/java/errorprone.xml/UnusedNullCheckInEquals" />
<rule ref="category/java/errorprone.xml/UselessOperationOnImmutable" /> <rule ref="category/java/errorprone.xml/UselessOperationOnImmutable" />
<rule ref="category/java/errorprone.xml/UnnecessaryBooleanAssertion" />
<rule ref="category/java/errorprone.xml/UnnecessaryCaseChange" />
<rule ref="category/java/multithreading.xml/AvoidThreadGroup" /> <rule ref="category/java/multithreading.xml/AvoidThreadGroup" />
<rule ref="category/java/multithreading.xml/DontCallThreadRun" /> <rule ref="category/java/multithreading.xml/DontCallThreadRun" />
<rule ref="category/java/multithreading.xml/DoubleCheckedLocking" /> <rule ref="category/java/multithreading.xml/DoubleCheckedLocking" />

2
src/main/java/com/sheepit/client/standalone/Worker.java
View File

@@ -406,7 +406,7 @@ public class Worker {
} }
if (shutdownMode != null) { if (shutdownMode != null) {
if (shutdownMode.toLowerCase().equals("wait") || shutdownMode.toLowerCase().equals("hard")) { if (shutdownMode.equalsIgnoreCase("wait") || shutdownMode.equalsIgnoreCase("hard")) {
config.setShutdownMode(shutdownMode.toLowerCase()); config.setShutdownMode(shutdownMode.toLowerCase());
} }
else { else {