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 2015/06/13 13:33:05 UTC

svn commit: r1685240 - /pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/color/PDColor.java

Author: tilman
Date: Sat Jun 13 11:33:05 2015
New Revision: 1685240

URL: http://svn.apache.org/r1685240
Log:
PDFBOX-2830: don't return pattern name in toCOSArray()

Modified:
    pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/color/PDColor.java

Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/color/PDColor.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/color/PDColor.java?rev=1685240&r1=1685239&r2=1685240&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/color/PDColor.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/color/PDColor.java Sat Jun 13 11:33:05 2015
@@ -47,7 +47,7 @@ public final class PDColor
     {
         if (array.size() > 0 && array.get(array.size() - 1) instanceof COSName)
         {
-            // color components (optional)
+            // color components (optional), for the color of an uncoloured tiling pattern
             components = new float[array.size() - 1];
             for (int i = 0; i < array.size() - 1; i++)
             {
@@ -153,14 +153,13 @@ public final class PDColor
     }
 
     /**
-     * Returns this color value as a COS array
-     * @return the color value as a COS array
+     * Returns the color component values as a COS array
+     * @return the color component values as a COS array
      */
     public COSArray toCOSArray()
     {
         COSArray array = new COSArray();
         array.setFloatArray(components);
-        array.add(patternName);
         return array;
     }