Author: tilman
Date: Tue Sep 16 12:46:47 2025
New Revision: 1928509
Log:
PDFBOX-5660: refactor, as suggested by Valery Bokov
Modified:
pdfbox/branches/3.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fdf/FDFAnnotation.java
pdfbox/branches/3.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fdf/FDFAnnotationCircle.java
pdfbox/branches/3.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fdf/FDFAnnotationLine.java
pdfbox/branches/3.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fdf/FDFAnnotationPolygon.java
pdfbox/branches/3.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fdf/FDFAnnotationPolyline.java
pdfbox/branches/3.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fdf/FDFAnnotationSquare.java
Modified:
pdfbox/branches/3.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fdf/FDFAnnotation.java
==============================================================================
---
pdfbox/branches/3.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fdf/FDFAnnotation.java
Tue Sep 16 12:45:32 2025 (r1928508)
+++
pdfbox/branches/3.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fdf/FDFAnnotation.java
Tue Sep 16 12:46:47 2025 (r1928509)
@@ -481,8 +481,13 @@ public abstract class FDFAnnotation impl
*/
public Color getColor()
{
+ return getColor(COSName.C);
+ }
+
+ final Color getColor(COSName colorName)
+ {
Color retval = null;
- COSArray array = annot.getCOSArray(COSName.C);
+ COSArray array = annot.getCOSArray(colorName);
if (array != null)
{
float[] rgb = array.toFloatArray();
Modified:
pdfbox/branches/3.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fdf/FDFAnnotationCircle.java
==============================================================================
---
pdfbox/branches/3.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fdf/FDFAnnotationCircle.java
Tue Sep 16 12:45:32 2025 (r1928508)
+++
pdfbox/branches/3.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fdf/FDFAnnotationCircle.java
Tue Sep 16 12:46:47 2025 (r1928509)
@@ -113,17 +113,7 @@ public class FDFAnnotationCircle extends
*/
public Color getInteriorColor()
{
- Color retval = null;
- COSArray array = annot.getCOSArray(COSName.IC);
- if (array != null)
- {
- float[] rgb = array.toFloatArray();
- if (rgb.length >= 3)
- {
- retval = new Color(rgb[0], rgb[1], rgb[2]);
- }
- }
- return retval;
+ return getColor(COSName.IC);
}
/**
Modified:
pdfbox/branches/3.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fdf/FDFAnnotationLine.java
==============================================================================
---
pdfbox/branches/3.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fdf/FDFAnnotationLine.java
Tue Sep 16 12:45:32 2025 (r1928508)
+++
pdfbox/branches/3.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fdf/FDFAnnotationLine.java
Tue Sep 16 12:46:47 2025 (r1928509)
@@ -258,17 +258,7 @@ public class FDFAnnotationLine extends F
*/
public Color getInteriorColor()
{
- Color retval = null;
- COSArray array = annot.getCOSArray(COSName.IC);
- if (array != null)
- {
- float[] rgb = array.toFloatArray();
- if (rgb.length >= 3)
- {
- retval = new Color(rgb[0], rgb[1], rgb[2]);
- }
- }
- return retval;
+ return getColor(COSName.IC);
}
/**
Modified:
pdfbox/branches/3.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fdf/FDFAnnotationPolygon.java
==============================================================================
---
pdfbox/branches/3.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fdf/FDFAnnotationPolygon.java
Tue Sep 16 12:45:32 2025 (r1928508)
+++
pdfbox/branches/3.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fdf/FDFAnnotationPolygon.java
Tue Sep 16 12:46:47 2025 (r1928509)
@@ -151,16 +151,6 @@ public class FDFAnnotationPolygon extend
*/
public Color getInteriorColor()
{
- Color retval = null;
- COSArray array = annot.getCOSArray(COSName.IC);
- if (array != null)
- {
- float[] rgb = array.toFloatArray();
- if (rgb.length >= 3)
- {
- retval = new Color(rgb[0], rgb[1], rgb[2]);
- }
- }
- return retval;
+ return getColor(COSName.IC);
}
}
Modified:
pdfbox/branches/3.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fdf/FDFAnnotationPolyline.java
==============================================================================
---
pdfbox/branches/3.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fdf/FDFAnnotationPolyline.java
Tue Sep 16 12:45:32 2025 (r1928508)
+++
pdfbox/branches/3.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fdf/FDFAnnotationPolyline.java
Tue Sep 16 12:46:47 2025 (r1928509)
@@ -234,16 +234,6 @@ public class FDFAnnotationPolyline exten
*/
public Color getInteriorColor()
{
- Color retval = null;
- COSArray array = annot.getCOSArray(COSName.IC);
- if (array != null)
- {
- float[] rgb = array.toFloatArray();
- if (rgb.length >= 3)
- {
- retval = new Color(rgb[0], rgb[1], rgb[2]);
- }
- }
- return retval;
+ return getColor(COSName.IC);
}
}
Modified:
pdfbox/branches/3.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fdf/FDFAnnotationSquare.java
==============================================================================
---
pdfbox/branches/3.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fdf/FDFAnnotationSquare.java
Tue Sep 16 12:45:32 2025 (r1928508)
+++
pdfbox/branches/3.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fdf/FDFAnnotationSquare.java
Tue Sep 16 12:46:47 2025 (r1928509)
@@ -114,17 +114,7 @@ public class FDFAnnotationSquare extends
*/
public Color getInteriorColor()
{
- Color retval = null;
- COSArray array = annot.getCOSArray(COSName.IC);
- if (array != null)
- {
- float[] rgb = array.toFloatArray();
- if (rgb.length >= 3)
- {
- retval = new Color(rgb[0], rgb[1], rgb[2]);
- }
- }
- return retval;
+ return getColor(COSName.IC);
}
/**
