Remove Unused local variable and add checkstyle rule about that.

eclipse-leshan · Oct 6, 2022 · 140f8b8 · 140f8b8
1 parent 2fdf9f1
commit 140f8b8
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 2 deletions.
diff --git a/build-config/pom.xml b/build-config/pom.xml
@@ -119,6 +119,7 @@ Contributors:
         <artifactId>maven-checkstyle-plugin</artifactId>
         <configuration>
           <configLocation>../eclipse/checkstyle.xml</configLocation>
+          <includeTestSourceDirectory>true</includeTestSourceDirectory>
           <consoleOutput>true</consoleOutput>
           <failsOnError>true</failsOnError>
           <linkXRef>false</linkXRef>

diff --git a/eclipse/checkstyle.xml b/eclipse/checkstyle.xml
@@ -12,13 +12,14 @@
             <property name="message" value="No `sysout` or `syserr` allowed."/>
         </module>
         <module name="AvoidStarImport" />
+        <module name="UnusedLocalVariable"/>
     </module>
 
     <module name="RegexpSingleline">
         <property name="format" value="\s+$"/>
         <property name="message" value="No trailing white space allowed."/>
     </module>
-    
+
     <!-- Remove some rules for specifics classes -->
     <module name="SuppressionSingleFilter">
         <property name="id" value="no_sys"/>

diff --git a/leshan-core/src/test/java/org/eclipse/leshan/core/node/codec/LwM2mNodeDecoderTest.java b/leshan-core/src/test/java/org/eclipse/leshan/core/node/codec/LwM2mNodeDecoderTest.java
@@ -1337,7 +1337,6 @@ public void senml_multiple_timestamped_nodes() throws CodecException {
                 .put(timestamp.plusSeconds(2), new LwM2mPath("/4/0/1"), LwM2mSingleResource.newIntegerResource(1, 2))
                 .put(timestamp.plusSeconds(3), new LwM2mPath("/4/0/1"), LwM2mSingleResource.newIntegerResource(1, 3));
 
-        TimestampedLwM2mNodes res = expectedResult.build();
         assertEquals(expectedResult.build(), data);
     }