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 2021/02/21 12:31:46 UTC
svn commit: r1886753 - in /pdfbox/trunk/pdfbox/src:
main/java/org/apache/pdfbox/cos/COSNumber.java
test/java/org/apache/pdfbox/cos/TestCOSStream.java
Author: tilman
Date: Sun Feb 21 12:31:45 2021
New Revision: 1886753
URL: http://svn.apache.org/viewvc?rev=1886753&view=rev
Log:
PDFBOX-4892: Sonar fix / revert
Modified:
pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/cos/COSNumber.java
pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/cos/TestCOSStream.java
Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/cos/COSNumber.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/cos/COSNumber.java?rev=1886753&r1=1886752&r2=1886753&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/cos/COSNumber.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/cos/COSNumber.java Sun Feb 21 12:31:45 2021
@@ -62,7 +62,7 @@ public abstract class COSNumber extends
char digit = number.charAt(0);
if ('0' <= digit && digit <= '9')
{
- return COSInteger.get(digit - '0');
+ return COSInteger.get((long) digit - '0');
}
else if (digit == '-' || digit == '.')
{
Modified: pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/cos/TestCOSStream.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/cos/TestCOSStream.java?rev=1886753&r1=1886752&r2=1886753&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/cos/TestCOSStream.java (original)
+++ pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/cos/TestCOSStream.java Sun Feb 21 12:31:45 2021
@@ -167,7 +167,7 @@ class TestCOSStream
try (COSStream stream = new COSStream())
{
assertFalse(stream.hasData());
- Assertions.assertThrows(IOException.class, () -> stream.createInputStream().close(),
+ Assertions.assertThrows(IOException.class, () -> stream.createInputStream(),
"createInputStream should have thrown an IOException");
byte[] testString = "This is a test string to be used as input for TestCOSStream"