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 2020/12/23 12:40:52 UTC
svn commit: r1884750 -
/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/text/PDFTextStripper.java
Author: tilman
Date: Wed Dec 23 12:40:52 2020
New Revision: 1884750
URL: http://svn.apache.org/viewvc?rev=1884750&view=rev
Log:
PDFBOX-5055: remove unneeded null check (assigning null is possible, but shouldn't be done, and maybe this field shouldn't be protected)
Modified:
pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/text/PDFTextStripper.java
Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/text/PDFTextStripper.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/text/PDFTextStripper.java?rev=1884750&r1=1884749&r2=1884750&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/text/PDFTextStripper.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/text/PDFTextStripper.java Wed Dec 23 12:40:52 2020
@@ -207,10 +207,7 @@ public class PDFTextStripper extends Leg
{
currentPageNo = 0;
document = null;
- if (charactersByArticle != null)
- {
- charactersByArticle.clear();
- }
+ charactersByArticle.clear();
characterListMapping.clear();
}