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/06/30 08:57:03 UTC

svn commit: r1862334 - /pdfbox/branches/issue4569/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/encryption/SaslPrep.java

Author: tilman
Date: Sun Jun 30 08:57:03 2019
New Revision: 1862334

URL: http://svn.apache.org/viewvc?rev=1862334&view=rev
Log:
PDFBOX-4071: Sonar fix: don't assign loop counter from within the loop body

Modified:
    pdfbox/branches/issue4569/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/encryption/SaslPrep.java

Modified: pdfbox/branches/issue4569/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/encryption/SaslPrep.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/issue4569/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/encryption/SaslPrep.java?rev=1862334&r1=1862333&r2=1862334&view=diff
==============================================================================
--- pdfbox/branches/issue4569/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/encryption/SaslPrep.java (original)
+++ pdfbox/branches/issue4569/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/encryption/SaslPrep.java Sun Jun 30 08:57:03 2019
@@ -96,7 +96,8 @@ class SaslPrep
         boolean containsRandALCat = false;
         boolean containsLCat = false;
         boolean initialRandALCat = false;
-        for (int i = 0; i < normalized.length();)
+        int i = 0;
+        while (i < normalized.length())
         {
             final int codepoint = normalized.codePointAt(i);
             // 3. Prohibit