Author: tilman
Date: Mon Nov 17 14:09:43 2025
New Revision: 1929805
Log:
PDFBOX-5660: avoid NPE, as suggested by Valery Bokov
Modified:
pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/annotation/handlers/PDLineAppearanceHandler.java
Modified:
pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/annotation/handlers/PDLineAppearanceHandler.java
==============================================================================
---
pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/annotation/handlers/PDLineAppearanceHandler.java
Mon Nov 17 12:45:02 2025 (r1929804)
+++
pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/annotation/handlers/PDLineAppearanceHandler.java
Mon Nov 17 14:09:43 2025 (r1929805)
@@ -174,7 +174,7 @@ public class PDLineAppearanceHandler ext
float contentLength = 0;
try
{
- contentLength =
font.getStringWidth(annotation.getContents()) / 1000 * FONT_SIZE;
+ contentLength = font.getStringWidth(contents) / 1000 *
FONT_SIZE;
//TODO How to decide the size of the font?
// 9 seems to be standard, but if the text doesn't fit, a
scaling is done
@@ -183,7 +183,7 @@ public class PDLineAppearanceHandler ext
catch (IllegalArgumentException ex)
{
// Adobe Reader displays placeholders instead
- LOG.error("line text '" + annotation.getContents() + "'
can't be shown", ex);
+ LOG.error("line text '" + contents + "' can't be shown",
ex);
}
float xOffset = (lineLength - contentLength) / 2;
float yOffset;
@@ -235,7 +235,7 @@ public class PDLineAppearanceHandler ext
cs.setFont(font, FONT_SIZE);
cs.newLineAtOffset(xOffset + captionHorizontalOffset,
y + yOffset + captionVerticalOffset);
- cs.showText(annotation.getContents());
+ cs.showText(contents);
cs.endText();
}