You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by ti...@apache.org on 2019/12/07 06:01:55 UTC

svn commit: r1870955 - /pdfbox/branches/issue4569/pdfbox/src/main/java/org/apache/pdfbox/util/DateConverter.java

Author: tilman
Date: Sat Dec  7 06:01:55 2019
New Revision: 1870955

URL: http://svn.apache.org/viewvc?rev=1870955&view=rev
Log:
PDFBOX-4071: remove unneeded null check that confuses Sonar

Modified:
    pdfbox/branches/issue4569/pdfbox/src/main/java/org/apache/pdfbox/util/DateConverter.java

Modified: pdfbox/branches/issue4569/pdfbox/src/main/java/org/apache/pdfbox/util/DateConverter.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/issue4569/pdfbox/src/main/java/org/apache/pdfbox/util/DateConverter.java?rev=1870955&r1=1870954&r2=1870955&view=diff
==============================================================================
--- pdfbox/branches/issue4569/pdfbox/src/main/java/org/apache/pdfbox/util/DateConverter.java (original)
+++ pdfbox/branches/issue4569/pdfbox/src/main/java/org/apache/pdfbox/util/DateConverter.java Sat Dec  7 06:01:55 2019
@@ -348,8 +348,9 @@ public final class DateConverter
      */
     private static char skipOptionals(String text, ParsePosition where, String optionals)
     {
-        char retval = ' ', currch;
-        while (text != null && where.getIndex() < text.length() &&
+        char retval = ' ';
+        char currch;
+        while (where.getIndex() < text.length() &&
                optionals.indexOf((currch = text.charAt(where.getIndex()))) >= 0)
         {
             retval = (currch != ' ') ? currch : retval;