Confusing statement renamed to isNotValid

Author: filipjonckers

src/main/java/aero/t2s/modes/decoder/Common.java

@@ -105,7 +105,7 @@ public static short[] getParity(short[] data) {
         return Arrays.copyOf(parity, CRC_POLYNOMIAL.length);
     }
 
-    public static boolean isValid(short[] data) {
+    public static boolean isNotValid(short[] data) {
         short[] payload = Arrays.copyOfRange(data, 0, data.length - 3);
         short[] parity = Arrays.copyOfRange(data, data.length - 3, data.length);
 

src/main/java/aero/t2s/modes/decoder/Decoder.java

@@ -3,10 +3,7 @@
 import aero.t2s.modes.Track;
 import aero.t2s.modes.database.ModeSDatabase;
 import aero.t2s.modes.decoder.df.*;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 
-import java.util.HashMap;
 import java.util.Map;
 
 public class Decoder {
@@ -24,8 +21,7 @@ public Decoder(Map<String, Track> tracks, double originLat, double originLon, Mo
     }
 
     public DownlinkFormat decode(short[] data) throws UnknownDownlinkFormatException {
-        // @Todo: Confusing statement this should probably be refactored to return inverse or renamed to invalid
-        if (Common.isValid(data)) {
+        if (Common.isNotValid(data)) {
             return null;
         }