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 2019/07/07 16:22:46 UTC
svn commit: r1862700 -
/pdfbox/branches/issue4569/pdfbox/src/main/java/org/apache/pdfbox/text/TextPosition.java
Author: tilman
Date: Sun Jul 7 16:22:46 2019
New Revision: 1862700
URL: http://svn.apache.org/viewvc?rev=1862700&view=rev
Log:
PDFBOX-4071: SonarQube fix: remove conditions that are always true
Modified:
pdfbox/branches/issue4569/pdfbox/src/main/java/org/apache/pdfbox/text/TextPosition.java
Modified: pdfbox/branches/issue4569/pdfbox/src/main/java/org/apache/pdfbox/text/TextPosition.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/issue4569/pdfbox/src/main/java/org/apache/pdfbox/text/TextPosition.java?rev=1862700&r1=1862699&r2=1862700&view=diff
==============================================================================
--- pdfbox/branches/issue4569/pdfbox/src/main/java/org/apache/pdfbox/text/TextPosition.java (original)
+++ pdfbox/branches/issue4569/pdfbox/src/main/java/org/apache/pdfbox/text/TextPosition.java Sun Jul 7 16:22:46 2019
@@ -602,20 +602,20 @@ public final class TextPosition
}
// diacritic completely covers this character and therefore we assume that this is the
// character the diacritic belongs to
- else if (diacXStart < currCharXStart && diacXEnd > currCharXEnd)
+ else if (diacXStart < currCharXStart)
{
insertDiacritic(i, diacritic);
wasAdded = true;
}
// otherwise, The diacritic modifies this character because its completely
// contained by the character width
- else if (diacXStart >= currCharXStart && diacXEnd <= currCharXEnd)
+ else if (diacXEnd <= currCharXEnd)
{
insertDiacritic(i, diacritic);
wasAdded = true;
}
// last character in the TextPosition so we add diacritic to the end
- else if (diacXStart >= currCharXStart && diacXEnd > currCharXEnd && i == strLen - 1)
+ else if (i == strLen - 1)
{
insertDiacritic(i, diacritic);
wasAdded = true;