Skip to content

Commit dd276b2

Browse files
committed
Java 5 compatibility updates.
1 parent d3d4a3a commit dd276b2

2 files changed

Lines changed: 12 additions & 11 deletions

File tree

pg/src/main/java/org/bouncycastle/bcpg/ArmoredOutputStream.java

Lines changed: 9 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -622,7 +622,7 @@ public Builder addComment(String comment)
622622
*/
623623
private Builder setSingletonHeader(String key, String value)
624624
{
625-
if (value == null || value.trim().isEmpty())
625+
if (value == null || value.trim().length() == 0)
626626
{
627627
this.headers.remove(key);
628628
}
@@ -647,7 +647,7 @@ private Builder setSingletonHeader(String key, String value)
647647
*/
648648
private Builder addHeader(String key, String value)
649649
{
650-
if (value == null || value.trim().isEmpty())
650+
if (value == null || value.trim().length() == 0)
651651
{
652652
return this;
653653
}
@@ -663,11 +663,12 @@ private Builder addHeader(String key, String value)
663663
String trimmed = value.trim();
664664
for (String line : trimmed.split("\n"))
665665
{
666-
if (line.trim().isEmpty())
666+
String lineTrim = line.trim();
667+
if (lineTrim.length() == 0)
667668
{
668669
continue;
669670
}
670-
values.add(line.trim());
671+
values.add(lineTrim);
671672
}
672673
return this;
673674
}
@@ -683,7 +684,7 @@ private Builder addHeader(String key, String value)
683684
*/
684685
private Builder replaceHeader(String key, String value)
685686
{
686-
if (value == null || value.trim().isEmpty())
687+
if (value == null || value.trim().length() == 0)
687688
{
688689
return this;
689690
}
@@ -694,11 +695,12 @@ private Builder replaceHeader(String key, String value)
694695
String trimmed = value.trim();
695696
for (String line : trimmed.split("\n"))
696697
{
697-
if (line.trim().isEmpty())
698+
String lineTrim = line.trim();
699+
if (lineTrim.length() == 0)
698700
{
699701
continue;
700702
}
701-
values.add(line.trim());
703+
values.add(lineTrim);
702704
}
703705

704706
headers.put(key, values);

prov/src/test/java/org/bouncycastle/jcajce/provider/test/CompositeSignaturesTest.java

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@
33
import java.io.BufferedReader;
44
import java.io.InputStream;
55
import java.io.InputStreamReader;
6-
import java.nio.charset.StandardCharsets;
76
import java.security.KeyFactory;
87
import java.security.KeyPair;
98
import java.security.KeyPairGenerator;
@@ -163,11 +162,11 @@ public void testSigningAndVerificationInternal()
163162
KeyPair keyPair = keyPairGenerator.generateKeyPair();
164163
Signature signature = Signature.getInstance(oid, "BC");
165164
signature.initSign(keyPair.getPrivate());
166-
signature.update(messageToBeSigned.getBytes(StandardCharsets.UTF_8));
165+
signature.update(Strings.toUTF8ByteArray(messageToBeSigned));
167166
byte[] signatureValue = signature.sign();
168167

169168
signature.initVerify(keyPair.getPublic());
170-
signature.update(messageToBeSigned.getBytes(StandardCharsets.UTF_8));
169+
signature.update(Strings.toUTF8ByteArray(messageToBeSigned));
171170
TestCase.assertTrue(signature.verify(signatureValue));
172171
}
173172
}
@@ -181,7 +180,7 @@ public void testDecodingAndVerificationExternal()
181180
int count = 0;
182181
while ((line = reader.readLine()) != null)
183182
{
184-
if (line.isEmpty())
183+
if (line.length() == 0)
185184
{
186185
continue;
187186
}

0 commit comments

Comments
 (0)