This is an automated email from the ASF dual-hosted git repository.
jamesbognar pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/juneau.git
The following commit(s) were added to refs/heads/master by this push:
new 4204c5204e SonarQube bug fixes
4204c5204e is described below
commit 4204c5204e6091180836ded6069cab0a29d6dae8
Author: James Bognar <[email protected]>
AuthorDate: Thu Feb 5 11:29:27 2026 -0500
SonarQube bug fixes
---
.../src/main/java/org/apache/juneau/commons/reflect/ClassInfo.java | 2 --
.../src/main/java/org/apache/juneau/commons/utils/StringUtils.java | 2 +-
.../src/main/java/org/apache/juneau/msgpack/MsgPackInputStream.java | 4 +++-
.../src/test/java/org/apache/juneau/a/rttests/RoundTripMaps_Test.java | 1 +
4 files changed, 5 insertions(+), 4 deletions(-)
diff --git
a/juneau-core/juneau-commons/src/main/java/org/apache/juneau/commons/reflect/ClassInfo.java
b/juneau-core/juneau-commons/src/main/java/org/apache/juneau/commons/reflect/ClassInfo.java
index 79332c0e6a..8cd7b751ab 100644
---
a/juneau-core/juneau-commons/src/main/java/org/apache/juneau/commons/reflect/ClassInfo.java
+++
b/juneau-core/juneau-commons/src/main/java/org/apache/juneau/commons/reflect/ClassInfo.java
@@ -1412,10 +1412,8 @@ public class ClassInfo extends ElementInfo implements
Annotatable, Type, Compara
if (gct instanceof ParameterizedType gct2)
return
Array.newInstance((Class<?>)gct2.getRawType(), 0).getClass();
} else if (actualType instanceof TypeVariable<?> actualType3) {
- var nestedOuterTypes = new LinkedList<Class<?>>();
for (Class<?> ec = cc.getEnclosingClass(); nn(ec); ec =
ec.getEnclosingClass()) {
var outerClass = ec;
- nestedOuterTypes.add(outerClass);
var outerTypeMap = new HashMap<Type,Type>();
extractTypes(outerTypeMap, outerClass);
for (var entry : outerTypeMap.entrySet()) {
diff --git
a/juneau-core/juneau-commons/src/main/java/org/apache/juneau/commons/utils/StringUtils.java
b/juneau-core/juneau-commons/src/main/java/org/apache/juneau/commons/utils/StringUtils.java
index 664f1cf7fa..60b63cec88 100644
---
a/juneau-core/juneau-commons/src/main/java/org/apache/juneau/commons/utils/StringUtils.java
+++
b/juneau-core/juneau-commons/src/main/java/org/apache/juneau/commons/utils/StringUtils.java
@@ -4927,7 +4927,7 @@ public class StringUtils {
public static String obfuscate(String s) {
if (s == null || s.length() < 2)
return "*";
- return s.substring(0, 1) + s.substring(1).replaceAll(".", "*");
+ return s.substring(0, 1) + s.substring(1).replaceAll(".", "*");
// NOSONAR - Dot matches any character, which is intended
}
/**
diff --git
a/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/msgpack/MsgPackInputStream.java
b/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/msgpack/MsgPackInputStream.java
index 1295e0ecbe..e13ce930b1 100644
---
a/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/msgpack/MsgPackInputStream.java
+++
b/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/msgpack/MsgPackInputStream.java
@@ -116,7 +116,9 @@ public class MsgPackInputStream extends ParserInputStream {
*/
byte[] readBinary() throws IOException {
var b = new byte[(int)length];
- read(b);
+ var bytesRead = read(b);
+ if (bytesRead != b.length)
+ throw new IOException("Expected to read " + b.length +
" bytes but only read " + bytesRead);
return b;
}
diff --git
a/juneau-utest/src/test/java/org/apache/juneau/a/rttests/RoundTripMaps_Test.java
b/juneau-utest/src/test/java/org/apache/juneau/a/rttests/RoundTripMaps_Test.java
index eafd72dfb4..6e40e7a69a 100755
---
a/juneau-utest/src/test/java/org/apache/juneau/a/rttests/RoundTripMaps_Test.java
+++
b/juneau-utest/src/test/java/org/apache/juneau/a/rttests/RoundTripMaps_Test.java
@@ -284,6 +284,7 @@ class RoundTripMaps_Test extends TestBase {
assertEquals("a", x.get(TestEnum.FOO));
assertNull(null, x.get(TestEnum.BAR));
+ @SuppressWarnings("java:S1640") // HashMap required - test
needs null key support which EnumMap doesn't allow
var x2 = new HashMap<TestEnum,String>();
x2.put(TestEnum.FOO, "a");
x2.put(TestEnum.BAR, null);
