Fix use of StringTokenizer methods 74/83674/1
authorRobert Varga <robert.varga@pantheon.tech>
Thu, 15 Aug 2019 11:21:56 +0000 (13:21 +0200)
committerRobert Varga <robert.varga@pantheon.tech>
Thu, 15 Aug 2019 11:49:48 +0000 (13:49 +0200)
Rather than using Enumeration-related methods (*Element()), use
the methods provided by StringTokenizer (*Token()), which have
the same functionality, but are not down-casted to Object.

Change-Id: I5c0c258db8e70cafef058d628c20bf9cbeb26cb0
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
(cherry picked from commit fe27f4e384d591021663abbcbfb505cc280c093b)

binding/mdsal-binding-java-api-generator/src/main/java/org/opendaylight/mdsal/binding/java/api/generator/BaseTemplate.xtend

index fae62150fd5c81caf66773946027bb4fefff4ed3..d5de5a82ba15bb5daae8b0cd5479ae0289ace197 100644 (file)
@@ -368,8 +368,8 @@ abstract class BaseTemplate extends JavaFileTemplate {
 
         val StringTokenizer tokenizer = new StringTokenizer(formattedText, " ", true);
 
-        while (tokenizer.hasMoreElements) {
-            val nextElement = tokenizer.nextElement.toString
+        while (tokenizer.hasMoreTokens) {
+            val nextElement = tokenizer.nextToken
 
             if (lineBuilder.length != 0 && lineBuilder.length + nextElement.length > 80) {
                 // FIXME: what tricks are we playing here? Equality probably does not trigger ever
@@ -386,7 +386,7 @@ abstract class BaseTemplate extends JavaFileTemplate {
                 sb.append(lineBuilder).append(NEW_LINE)
                 lineBuilder.setLength(0)
 
-                if (nextElement.toString == ' ') {
+                if (nextElement == " ") {
                     isFirstElementOnNewLineEmptyChar = !isFirstElementOnNewLineEmptyChar;
                 }
             }