This is an automated email from the ASF dual-hosted git repository.
garydgregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-codec.git
The following commit(s) were added to refs/heads/master by this push:
new 3351e631 Add messages when throwing NullPointerException.
3351e631 is described below
commit 3351e63155fa41fbfcb0744efa960adce1529798
Author: Gary Gregory <[email protected]>
AuthorDate: Wed Jul 22 10:53:02 2026 -0400
Add messages when throwing NullPointerException.
---
src/changes/changes.xml | 1 +
src/main/java/org/apache/commons/codec/cli/Digest.java | 2 +-
src/main/java/org/apache/commons/codec/digest/Blake3.java | 6 +++---
src/main/java/org/apache/commons/codec/digest/GitIdentifiers.java | 8 ++++----
4 files changed, 9 insertions(+), 8 deletions(-)
diff --git a/src/changes/changes.xml b/src/changes/changes.xml
index 6183f486..06f16bc0 100644
--- a/src/changes/changes.xml
+++ b/src/changes/changes.xml
@@ -54,6 +54,7 @@ The <action> type attribute can be add,update,fix,remove.
<action type="fix" issue="CODEC-338" dev="ggregory" due-to="Ruiqi Dong,
Gary Gregory">PercentCodec loses literal '+' when plusForSpace is
enabled.</action>
<action type="fix" issue="CODEC-337" dev="pkarwasz" due-to="Ruiqi Dong,
Gary Gregory">Digest ALL reuses System.in, so only the first algorithm sees the
real input (#431).</action>
<action type="fix" dev="ggregory" due-to="Gary Gregory">Fix
Base64.toIntegerBytes(BigInteger) for zero edge case (#441).</action>
+ <action type="fix" dev="ggregory" due-to="Gary Gregory">Add messages
when throwing NullPointerException.</action>
<!-- ADD -->
<!-- UPDATE -->
<action type="update" dev="ggregory" due-to="Gary Gregory">Bump
org.apache.commons:commons-parent from 98 to 103.</action>
diff --git a/src/main/java/org/apache/commons/codec/cli/Digest.java
b/src/main/java/org/apache/commons/codec/cli/Digest.java
index 7e1f5283..91f3651d 100644
--- a/src/main/java/org/apache/commons/codec/cli/Digest.java
+++ b/src/main/java/org/apache/commons/codec/cli/Digest.java
@@ -60,7 +60,7 @@ public class Digest {
private final String[] inputs;
private Digest(final String[] args) {
- Objects.requireNonNull(args);
+ Objects.requireNonNull(args, "args");
final int argsLength = args.length;
if (argsLength == 0) {
throw new IllegalArgumentException(String.format("Usage: java %s
[algorithm] [FILE|DIRECTORY|string] ...", Digest.class.getName()));
diff --git a/src/main/java/org/apache/commons/codec/digest/Blake3.java
b/src/main/java/org/apache/commons/codec/digest/Blake3.java
index a2af6222..967fe3d5 100644
--- a/src/main/java/org/apache/commons/codec/digest/Blake3.java
+++ b/src/main/java/org/apache/commons/codec/digest/Blake3.java
@@ -290,7 +290,7 @@ public final class Blake3 {
// @formatter:on
private static void checkBufferArgs(final byte[] buffer, final int offset,
final int length) {
- Objects.requireNonNull(buffer);
+ Objects.requireNonNull(buffer, "buffer");
if (offset < 0) {
throw new IndexOutOfBoundsException("Offset must be non-negative");
}
@@ -365,7 +365,7 @@ public final class Blake3 {
* @throws NullPointerException if kdfContext is null.
*/
public static Blake3 initKeyDerivationFunction(final byte[] kdfContext) {
- Objects.requireNonNull(kdfContext);
+ Objects.requireNonNull(kdfContext, "kdfContext");
final EngineState kdf = new EngineState(IV, DERIVE_KEY_CONTEXT);
kdf.inputData(kdfContext, 0, kdfContext.length);
final byte[] key = new byte[KEY_LEN];
@@ -383,7 +383,7 @@ public final class Blake3 {
* @throws IllegalArgumentException if key is not 32 bytes.
*/
public static Blake3 initKeyedHash(final byte[] key) {
- Objects.requireNonNull(key);
+ Objects.requireNonNull(key, "key");
if (key.length != KEY_LEN) {
throw new IllegalArgumentException("Blake3 keys must be 32 bytes");
}
diff --git a/src/main/java/org/apache/commons/codec/digest/GitIdentifiers.java
b/src/main/java/org/apache/commons/codec/digest/GitIdentifiers.java
index 73e8a4ab..0f7a3015 100644
--- a/src/main/java/org/apache/commons/codec/digest/GitIdentifiers.java
+++ b/src/main/java/org/apache/commons/codec/digest/GitIdentifiers.java
@@ -95,13 +95,13 @@ public class GitIdentifiers {
* @param rawObjectId The id of the entry, not null.
*/
DirectoryEntry(final String name, final FileMode type, final byte[]
rawObjectId) {
- if (Objects.requireNonNull(name).indexOf('/') >= 0) {
+ if (Objects.requireNonNull(name, "name").indexOf('/') >= 0) {
throw new IllegalArgumentException("Entry name must not
contain '/': " + name);
}
this.name = name;
- this.type = Objects.requireNonNull(type);
+ this.type = Objects.requireNonNull(type, "type");
this.sortKey = type == FileMode.DIRECTORY ? name + "/" : name;
- this.rawObjectId = Objects.requireNonNull(rawObjectId);
+ this.rawObjectId = Objects.requireNonNull(rawObjectId,
"rawObjectId");
}
@Override
@@ -223,7 +223,7 @@ public class GitIdentifiers {
private final MessageDigest messageDigest;
private TreeIdBuilder(final MessageDigest messageDigest) {
- this.messageDigest = Objects.requireNonNull(messageDigest);
+ this.messageDigest = Objects.requireNonNull(messageDigest,
"messageDigest");
}
/**
