From 9e499dad6d92aeee4810446c2c286155149bc5e2 Mon Sep 17 00:00:00 2001 From: David Pursehouse Date: Sun, 16 Jun 2019 13:35:35 +0900 Subject: [PATCH] PacketLineIn: Suppress comparison warnings for END and DELIM Reference comparison is intentional. The END and DELIM string constants are used as sentinels and will always be the same instances. Suppress both ReferenceEquality and StringEquality warnings. Change-Id: I4ce0495702c56b3911f42f26c2f81d28073cbe19 Signed-off-by: David Pursehouse --- .../src/org/eclipse/jgit/transport/PacketLineIn.java | 2 ++ 1 file changed, 2 insertions(+) diff --git a/org.eclipse.jgit/src/org/eclipse/jgit/transport/PacketLineIn.java b/org.eclipse.jgit/src/org/eclipse/jgit/transport/PacketLineIn.java index 90f1b373b..d73e1939a 100644 --- a/org.eclipse.jgit/src/org/eclipse/jgit/transport/PacketLineIn.java +++ b/org.eclipse.jgit/src/org/eclipse/jgit/transport/PacketLineIn.java @@ -259,6 +259,7 @@ public class PacketLineIn { * @return true if the given string is {@link #DELIM}, otherwise false. * @since 5.4 */ + @SuppressWarnings({ "ReferenceEquality", "StringEquality" }) public static boolean isDelimiter(String s) { return s == DELIM; } @@ -293,6 +294,7 @@ public class PacketLineIn { * @return true if the given string is {@link #END}, otherwise false. * @since 5.4 */ + @SuppressWarnings({ "ReferenceEquality", "StringEquality" }) public static boolean isEnd(String s) { return s == END; }