diff --git a/gradlew.bat b/gradlew.bat
index 6689b85b..93e3f59f 100644
--- a/gradlew.bat
+++ b/gradlew.bat
@@ -1,92 +1,92 @@
-@rem
-@rem Copyright 2015 the original author or authors.
-@rem
-@rem Licensed under the Apache License, Version 2.0 (the "License");
-@rem you may not use this file except in compliance with the License.
-@rem You may obtain a copy of the License at
-@rem
-@rem https://www.apache.org/licenses/LICENSE-2.0
-@rem
-@rem Unless required by applicable law or agreed to in writing, software
-@rem distributed under the License is distributed on an "AS IS" BASIS,
-@rem WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-@rem See the License for the specific language governing permissions and
-@rem limitations under the License.
-@rem
-
-@if "%DEBUG%"=="" @echo off
-@rem ##########################################################################
-@rem
-@rem Gradle startup script for Windows
-@rem
-@rem ##########################################################################
-
-@rem Set local scope for the variables with windows NT shell
-if "%OS%"=="Windows_NT" setlocal
-
-set DIRNAME=%~dp0
-if "%DIRNAME%"=="" set DIRNAME=.
-@rem This is normally unused
-set APP_BASE_NAME=%~n0
-set APP_HOME=%DIRNAME%
-
-@rem Resolve any "." and ".." in APP_HOME to make it shorter.
-for %%i in ("%APP_HOME%") do set APP_HOME=%%~fi
-
-@rem Add default JVM options here. You can also use JAVA_OPTS and GRADLE_OPTS to pass JVM options to this script.
-set DEFAULT_JVM_OPTS="-Xmx64m" "-Xms64m"
-
-@rem Find java.exe
-if defined JAVA_HOME goto findJavaFromJavaHome
-
-set JAVA_EXE=java.exe
-%JAVA_EXE% -version >NUL 2>&1
-if %ERRORLEVEL% equ 0 goto execute
-
-echo.
-echo ERROR: JAVA_HOME is not set and no 'java' command could be found in your PATH.
-echo.
-echo Please set the JAVA_HOME variable in your environment to match the
-echo location of your Java installation.
-
-goto fail
-
-:findJavaFromJavaHome
-set JAVA_HOME=%JAVA_HOME:"=%
-set JAVA_EXE=%JAVA_HOME%/bin/java.exe
-
-if exist "%JAVA_EXE%" goto execute
-
-echo.
-echo ERROR: JAVA_HOME is set to an invalid directory: %JAVA_HOME%
-echo.
-echo Please set the JAVA_HOME variable in your environment to match the
-echo location of your Java installation.
-
-goto fail
-
-:execute
-@rem Setup the command line
-
-set CLASSPATH=%APP_HOME%\gradle\wrapper\gradle-wrapper.jar
-
-
-@rem Execute Gradle
-"%JAVA_EXE%" %DEFAULT_JVM_OPTS% %JAVA_OPTS% %GRADLE_OPTS% "-Dorg.gradle.appname=%APP_BASE_NAME%" -classpath "%CLASSPATH%" org.gradle.wrapper.GradleWrapperMain %*
-
-:end
-@rem End local scope for the variables with windows NT shell
-if %ERRORLEVEL% equ 0 goto mainEnd
-
-:fail
-rem Set variable GRADLE_EXIT_CONSOLE if you need the _script_ return code instead of
-rem the _cmd.exe /c_ return code!
-set EXIT_CODE=%ERRORLEVEL%
-if %EXIT_CODE% equ 0 set EXIT_CODE=1
-if not ""=="%GRADLE_EXIT_CONSOLE%" exit %EXIT_CODE%
-exit /b %EXIT_CODE%
-
-:mainEnd
-if "%OS%"=="Windows_NT" endlocal
-
-:omega
+@rem
+@rem Copyright 2015 the original author or authors.
+@rem
+@rem Licensed under the Apache License, Version 2.0 (the "License");
+@rem you may not use this file except in compliance with the License.
+@rem You may obtain a copy of the License at
+@rem
+@rem https://www.apache.org/licenses/LICENSE-2.0
+@rem
+@rem Unless required by applicable law or agreed to in writing, software
+@rem distributed under the License is distributed on an "AS IS" BASIS,
+@rem WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+@rem See the License for the specific language governing permissions and
+@rem limitations under the License.
+@rem
+
+@if "%DEBUG%"=="" @echo off
+@rem ##########################################################################
+@rem
+@rem Gradle startup script for Windows
+@rem
+@rem ##########################################################################
+
+@rem Set local scope for the variables with windows NT shell
+if "%OS%"=="Windows_NT" setlocal
+
+set DIRNAME=%~dp0
+if "%DIRNAME%"=="" set DIRNAME=.
+@rem This is normally unused
+set APP_BASE_NAME=%~n0
+set APP_HOME=%DIRNAME%
+
+@rem Resolve any "." and ".." in APP_HOME to make it shorter.
+for %%i in ("%APP_HOME%") do set APP_HOME=%%~fi
+
+@rem Add default JVM options here. You can also use JAVA_OPTS and GRADLE_OPTS to pass JVM options to this script.
+set DEFAULT_JVM_OPTS="-Xmx64m" "-Xms64m"
+
+@rem Find java.exe
+if defined JAVA_HOME goto findJavaFromJavaHome
+
+set JAVA_EXE=java.exe
+%JAVA_EXE% -version >NUL 2>&1
+if %ERRORLEVEL% equ 0 goto execute
+
+echo.
+echo ERROR: JAVA_HOME is not set and no 'java' command could be found in your PATH.
+echo.
+echo Please set the JAVA_HOME variable in your environment to match the
+echo location of your Java installation.
+
+goto fail
+
+:findJavaFromJavaHome
+set JAVA_HOME=%JAVA_HOME:"=%
+set JAVA_EXE=%JAVA_HOME%/bin/java.exe
+
+if exist "%JAVA_EXE%" goto execute
+
+echo.
+echo ERROR: JAVA_HOME is set to an invalid directory: %JAVA_HOME%
+echo.
+echo Please set the JAVA_HOME variable in your environment to match the
+echo location of your Java installation.
+
+goto fail
+
+:execute
+@rem Setup the command line
+
+set CLASSPATH=%APP_HOME%\gradle\wrapper\gradle-wrapper.jar
+
+
+@rem Execute Gradle
+"%JAVA_EXE%" %DEFAULT_JVM_OPTS% %JAVA_OPTS% %GRADLE_OPTS% "-Dorg.gradle.appname=%APP_BASE_NAME%" -classpath "%CLASSPATH%" org.gradle.wrapper.GradleWrapperMain %*
+
+:end
+@rem End local scope for the variables with windows NT shell
+if %ERRORLEVEL% equ 0 goto mainEnd
+
+:fail
+rem Set variable GRADLE_EXIT_CONSOLE if you need the _script_ return code instead of
+rem the _cmd.exe /c_ return code!
+set EXIT_CODE=%ERRORLEVEL%
+if %EXIT_CODE% equ 0 set EXIT_CODE=1
+if not ""=="%GRADLE_EXIT_CONSOLE%" exit %EXIT_CODE%
+exit /b %EXIT_CODE%
+
+:mainEnd
+if "%OS%"=="Windows_NT" endlocal
+
+:omega
diff --git a/src/main/java/analyzer/exercises/hamming/HammingAnalyzer.java b/src/main/java/analyzer/exercises/hamming/HammingAnalyzer.java
index 9739d4cb..9d75df7a 100644
--- a/src/main/java/analyzer/exercises/hamming/HammingAnalyzer.java
+++ b/src/main/java/analyzer/exercises/hamming/HammingAnalyzer.java
@@ -57,10 +57,6 @@ public void analyze(Solution solution, Analysis analysis) {
analysis.addComment(new CalculateDistanceInConstructor());
}
- if (!walker.usesStringIsEmpty()) {
- analysis.addComment(new ShouldUseStringIsEmpty());
- }
-
if (walker.shouldUseStreamFilterAndCount()) {
analysis.addComment(new ShouldUseStreamFilterAndCount());
}
diff --git a/src/main/java/analyzer/exercises/hamming/HammingWalker.java b/src/main/java/analyzer/exercises/hamming/HammingWalker.java
index 25180374..5d01ca18 100644
--- a/src/main/java/analyzer/exercises/hamming/HammingWalker.java
+++ b/src/main/java/analyzer/exercises/hamming/HammingWalker.java
@@ -279,10 +279,6 @@ public boolean usesCharacterLiterals() {
return !hammingClass.findAll(CharLiteralExpr.class).isEmpty();
}
- public boolean usesStringIsEmpty() {
- return usesMethod("isEmpty");
- }
-
public boolean usesStringCharAtOrCodePointAt() {
return usesMethod("charAt") || usesMethod("codePointAt");
}
diff --git a/src/main/java/analyzer/exercises/hamming/ShouldUseStringIsEmpty.java b/src/main/java/analyzer/exercises/hamming/ShouldUseStringIsEmpty.java
deleted file mode 100644
index 2595ea02..00000000
--- a/src/main/java/analyzer/exercises/hamming/ShouldUseStringIsEmpty.java
+++ /dev/null
@@ -1,13 +0,0 @@
-package analyzer.exercises.hamming;
-
-import analyzer.Comment;
-
-/**
- * @see Markdown Template
- */
-class ShouldUseStringIsEmpty extends Comment {
- @Override
- public String getKey() {
- return "java.hamming.should_use_string_is_empty";
- }
-}
diff --git a/src/test/java/analyzer/exercises/hamming/HammingAnalyzerTest.java b/src/test/java/analyzer/exercises/hamming/HammingAnalyzerTest.java
index 8be0033a..48a72b1f 100644
--- a/src/test/java/analyzer/exercises/hamming/HammingAnalyzerTest.java
+++ b/src/test/java/analyzer/exercises/hamming/HammingAnalyzerTest.java
@@ -25,7 +25,6 @@ private static Stream testCases() {
Arguments.of("MustUseCharAtOrCodePointAt.java.txt", new Comment[]{new MustUseStringCharAtOrCodePointAt()}),
Arguments.of("NestedValidation.java.txt", new Comment[]{new CalculateDistanceInConstructor()}),
Arguments.of("NestedCalculation.java.txt", new Comment[0]),
- Arguments.of("ShouldUseStringIsEmpty.java.txt", new Comment[]{new ShouldUseStringIsEmpty()}),
Arguments.of("OptimalWithCalculationInGetHammingDistance.java.txt", new Comment[]{new CalculateDistanceInConstructor()}),
Arguments.of("OptimalWithCalculationDelegatedFromGetHammingDistance.java.txt", new Comment[]{new CalculateDistanceInConstructor()}),
Arguments.of("ConstructorTooLong.java.txt", new Comment[]{new ConstructorTooLong("Hamming")}),
diff --git a/src/test/resources/analyzer/exercises/hamming/ShouldUseStringIsEmpty.java.txt b/src/test/resources/analyzer/exercises/hamming/ShouldUseStringIsEmpty.java.txt
deleted file mode 100644
index 37b222ed..00000000
--- a/src/test/resources/analyzer/exercises/hamming/ShouldUseStringIsEmpty.java.txt
+++ /dev/null
@@ -1,32 +0,0 @@
-import java.util.stream.IntStream;
-
-/** Optimal solution, but does not use String.isEmpty. */
-class Hamming {
- private final int hammingDistance;
-
- Hamming(String leftStrand, String rightStrand) {
- validateStrandsHaveEqualLength(leftStrand, rightStrand);
-
- hammingDistance = (int) IntStream.range(0, leftStrand.length())
- .filter(index -> leftStrand.charAt(index) != rightStrand.charAt(index))
- .count();
- }
-
- private void validateStrandsHaveEqualLength() {
- if (leftStrand.length() == rightStrand.length()) {
- return;
- }
- if (leftStrand.length() == 0) {
- throw new IllegalArgumentException("left strand must not be empty.");
- }
- if (rightStrand.length() == 0) {
- throw new IllegalArgumentException("right strand must not be empty.");
- }
- throw new IllegalArgumentException(
- "leftStrand and rightStrand must be of equal length.");
- }
-
- int getHammingDistance() {
- return hammingDistance;
- }
-}
\ No newline at end of file
diff --git a/tests/hamming/expected_analysis.json b/tests/hamming/expected_analysis.json
index 7e7cbaa6..36d33952 100644
--- a/tests/hamming/expected_analysis.json
+++ b/tests/hamming/expected_analysis.json
@@ -3,10 +3,6 @@
"comment": "java.hamming.calculate_distance_in_constructor",
"type": "actionable"
},
- {
- "comment": "java.hamming.should_use_string_is_empty",
- "type": "informative"
- },
{
"comment": "java.general.feedback_request",
"type": "informative"