You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by ja...@apache.org on 2015/07/14 22:39:38 UTC

svn commit: r1691077 - in /pdfbox/trunk/tools/src/main/java/org/apache/pdfbox/tools/pdfdebugger: streampane/Stream.java ui/Tree.java

Author: jahewson
Date: Tue Jul 14 20:39:38 2015
New Revision: 1691077

URL: http://svn.apache.org/r1691077
Log:
PDFBOX-2530: Comma separation for filter labels

Modified:
    pdfbox/trunk/tools/src/main/java/org/apache/pdfbox/tools/pdfdebugger/streampane/Stream.java
    pdfbox/trunk/tools/src/main/java/org/apache/pdfbox/tools/pdfdebugger/ui/Tree.java

Modified: pdfbox/trunk/tools/src/main/java/org/apache/pdfbox/tools/pdfdebugger/streampane/Stream.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/tools/src/main/java/org/apache/pdfbox/tools/pdfdebugger/streampane/Stream.java?rev=1691077&r1=1691076&r2=1691077&view=diff
==============================================================================
--- pdfbox/trunk/tools/src/main/java/org/apache/pdfbox/tools/pdfdebugger/streampane/Stream.java (original)
+++ pdfbox/trunk/tools/src/main/java/org/apache/pdfbox/tools/pdfdebugger/streampane/Stream.java Tue Jul 14 20:39:38 2015
@@ -91,10 +91,6 @@ class Stream
         COSBase filters = stream.getFilters();
         if (filters != null)
         {
-            if (sb.length() > 0)
-            {
-                sb.append(", ");
-            }
             if (filters instanceof COSName)
             {
                 sb.append(((COSName) filters).getName());
@@ -104,6 +100,10 @@ class Stream
                 COSArray filterArray = (COSArray) filters;
                 for (int i = 0; i < filterArray.size(); i++)
                 {
+                    if (i > 0)
+                    {
+                        sb.append(", ");
+                    }
                     sb.append(((COSName) filterArray.get(i)).getName());
                 }
             }

Modified: pdfbox/trunk/tools/src/main/java/org/apache/pdfbox/tools/pdfdebugger/ui/Tree.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/tools/src/main/java/org/apache/pdfbox/tools/pdfdebugger/ui/Tree.java?rev=1691077&r1=1691076&r2=1691077&view=diff
==============================================================================
--- pdfbox/trunk/tools/src/main/java/org/apache/pdfbox/tools/pdfdebugger/ui/Tree.java (original)
+++ pdfbox/trunk/tools/src/main/java/org/apache/pdfbox/tools/pdfdebugger/ui/Tree.java Tue Jul 14 20:39:38 2015
@@ -181,10 +181,6 @@ public class Tree extends JTree
         COSBase filters = cosStream.getFilters();
         if (filters != null)
         {
-            if (sb.length() > 0)
-            {
-                sb.append(", ");
-            }
             if (filters instanceof COSName)
             {
                 sb.append(((COSName) filters).getName());
@@ -194,13 +190,16 @@ public class Tree extends JTree
                 COSArray filterArray = (COSArray) filters;
                 for (int i = 0; i < filterArray.size(); i++)
                 {
+                    if (i > 0)
+                    {
+                        sb.append(", ");
+                    }
                     sb.append(((COSName) filterArray.get(i)).getName());
                 }
             }
         }
         return sb.toString();
     }
-    
 
     /**
      * Produce JMenuItem that saves unfiltered stream