Skip to content

Commit a9ccd97

Browse files
vivek-0509romani
authored andcommitted
Issue #18926: Re-enable 'EqualsWithItself' inspection
1 parent c33aba0 commit a9ccd97

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

config/intellij-idea-inspections.xml

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1767,9 +1767,8 @@
17671767
enabled_by_default="true"/>
17681768
<inspection_tool class="EqualsWhichDoesntCheckParameterClass" enabled="true" level="ERROR"
17691769
enabled_by_default="true"/>
1770-
<!-- temporary disabled until https://github.com/checkstyle/checkstyle/issues/18926 -->
1771-
<inspection_tool class="EqualsWithItself" enabled="false" level="ERROR"
1772-
enabled_by_default="false"/>
1770+
<inspection_tool class="EqualsWithItself" enabled="true" level="ERROR"
1771+
enabled_by_default="true"/>
17731772
<!-- there is false-positive, and rest cases are result of our loading modules by reflection
17741773
that throws Error. Error is not always jvm problem,
17751774
a bunch of libraries throw Error instead of Exception -->

src/test/java/com/puppycrawl/tools/checkstyle/api/LineColumnTest.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,8 @@ public class LineColumnTest {
3030

3131
@Test
3232
public void testCompareToBothEqual() {
33-
final int actual = new LineColumn(0, 0).compareTo(new LineColumn(0, 0));
33+
final LineColumn lineColumn = new LineColumn(0, 0);
34+
final int actual = lineColumn.compareTo(new LineColumn(0, 0));
3435
assertWithMessage("Invalid LineColumn comparing result")
3536
.that(actual)
3637
.isEqualTo(0);

0 commit comments

Comments
 (0)