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-imaging.git
The following commit(s) were added to refs/heads/master by this push:
new e26d7ce5 Add messages when throwing NullPointerException.
e26d7ce5 is described below
commit e26d7ce54dc063ae9106243d8e71497912138e44
Author: Gary Gregory <[email protected]>
AuthorDate: Wed Jul 22 11:09:02 2026 -0400
Add messages when throwing NullPointerException.
---
src/changes/changes.xml | 1 +
src/main/java/org/apache/commons/imaging/ImageFormats.java | 2 +-
.../org/apache/commons/imaging/bytesource/InputStreamByteSource.java | 2 +-
.../java/org/apache/commons/imaging/formats/png/PngImageMetadata.java | 2 +-
4 files changed, 4 insertions(+), 3 deletions(-)
diff --git a/src/changes/changes.xml b/src/changes/changes.xml
index c7092152..8af35ee5 100644
--- a/src/changes/changes.xml
+++ b/src/changes/changes.xml
@@ -52,6 +52,7 @@ The <action> type attribute can be add,update,fix,remove.
<action type="fix" dev="ggregory" due-to="Gary Gregory">Fix Javadoc
warnings.</action>
<action type="fix" dev="ggregory" due-to="alhuda, Gary Gregory">Fix int
overflow in PCX image buffer sizing, now throws AllocationRequestException
(#552).</action>
<action type="fix" dev="ggregory" due-to="alhuda, Gary Gregory">Fix int
overflow in Allocator.array/arrayList size checks (#551).</action>
+ <action type="fix" dev="ggregory" due-to="Gary Gregory">Add messages
when throwing NullPointerException.</action>
<!-- ADD -->
<action type="fix" dev="ggregory" due-to="Sean R, Gary Gregory">Add unit
tests (#553).</action>
<!-- UPDATE -->
diff --git a/src/main/java/org/apache/commons/imaging/ImageFormats.java
b/src/main/java/org/apache/commons/imaging/ImageFormats.java
index 1d3fa380..b06806d9 100644
--- a/src/main/java/org/apache/commons/imaging/ImageFormats.java
+++ b/src/main/java/org/apache/commons/imaging/ImageFormats.java
@@ -95,7 +95,7 @@ public enum ImageFormats implements ImageFormat {
private final String[] extensions;
ImageFormats(final String... extensions) {
- this.extensions = Objects.requireNonNull(extensions);
+ this.extensions = Objects.requireNonNull(extensions, "extensions");
}
@Override
diff --git
a/src/main/java/org/apache/commons/imaging/bytesource/InputStreamByteSource.java
b/src/main/java/org/apache/commons/imaging/bytesource/InputStreamByteSource.java
index c7970b16..1ad5cea5 100644
---
a/src/main/java/org/apache/commons/imaging/bytesource/InputStreamByteSource.java
+++
b/src/main/java/org/apache/commons/imaging/bytesource/InputStreamByteSource.java
@@ -41,7 +41,7 @@ final class InputStreamByteSource extends ByteSource {
private boolean triedNext;
Block(final byte[] bytes) {
- this.bytes = Objects.requireNonNull(bytes);
+ this.bytes = Objects.requireNonNull(bytes, "bytes");
}
Block getNext() throws IOException {
diff --git
a/src/main/java/org/apache/commons/imaging/formats/png/PngImageMetadata.java
b/src/main/java/org/apache/commons/imaging/formats/png/PngImageMetadata.java
index ea20b836..88e04789 100644
--- a/src/main/java/org/apache/commons/imaging/formats/png/PngImageMetadata.java
+++ b/src/main/java/org/apache/commons/imaging/formats/png/PngImageMetadata.java
@@ -52,7 +52,7 @@ public class PngImageMetadata implements ImageMetadata {
* @param exif EXIF metadata.
*/
public PngImageMetadata(final ImageMetadata textualInformation, final
TiffImageMetadata exif) {
- this.textualInformation = Objects.requireNonNull(textualInformation);
+ this.textualInformation = Objects.requireNonNull(textualInformation,
"textualInformation");
this.exif = exif;
}