[core] Remove @SuppressWarnings for parseContentDepthWithRoundtrip()

Also use the most specific return type in the method's signature.
igniterealtime · Nov 25, 2020 · 02c9058 · 02c9058
1 parent ca497a2
commit 02c9058
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/smack-core/src/main/java/org/jivesoftware/smack/util/PacketParserUtils.java b/smack-core/src/main/java/org/jivesoftware/smack/util/PacketParserUtils.java
@@ -299,7 +299,7 @@ public static CharSequence parseContentDepth(XmlPullParser parser, int depth)
      */
     public static CharSequence parseContentDepth(XmlPullParser parser, int depth, boolean fullNamespaces) throws XmlPullParserException, IOException {
         if (parser.supportsRoundtrip()) {
-            return parseContentDepthWithRoundtrip(parser, depth, fullNamespaces);
+            return parseContentDepthWithRoundtrip(parser, depth);
         } else {
             return parseContentDepthWithoutRoundtrip(parser, depth, fullNamespaces);
         }
@@ -368,8 +368,7 @@ private static CharSequence parseContentDepthWithoutRoundtrip(XmlPullParser pars
         return xml;
     }
 
-    @SuppressWarnings("UnusedVariable")
-    private static CharSequence parseContentDepthWithRoundtrip(XmlPullParser parser, int depth, boolean fullNamespaces)
+    private static XmlStringBuilder parseContentDepthWithRoundtrip(XmlPullParser parser, int depth)
                     throws XmlPullParserException, IOException {
         XmlStringBuilder sb = new XmlStringBuilder();
         XmlPullParser.Event event = parser.getEventType();