Skip to content

Commit acab40e

Browse files
committed
Inline Objects.requireNonNull()
1 parent d1e58ba commit acab40e

File tree

2 files changed

+18
-14
lines changed

2 files changed

+18
-14
lines changed

src/main/java/at/favre/lib/bytes/Bytes.java

Lines changed: 17 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,11 @@
2121

2222
package at.favre.lib.bytes;
2323

24-
import java.io.*;
24+
import java.io.ByteArrayInputStream;
25+
import java.io.DataInput;
26+
import java.io.File;
27+
import java.io.InputStream;
28+
import java.io.Serializable;
2529
import java.math.BigInteger;
2630
import java.nio.ByteBuffer;
2731
import java.nio.ByteOrder;
@@ -31,7 +35,14 @@
3135
import java.nio.charset.StandardCharsets;
3236
import java.security.SecureRandom;
3337
import java.text.Normalizer;
34-
import java.util.*;
38+
import java.util.Arrays;
39+
import java.util.BitSet;
40+
import java.util.Collection;
41+
import java.util.Iterator;
42+
import java.util.List;
43+
import java.util.Objects;
44+
import java.util.Random;
45+
import java.util.UUID;
3546

3647
/**
3748
* Bytes is wrapper class for an byte-array that allows a lot of convenience operations on it:
@@ -501,8 +512,7 @@ public static Bytes from(char[] charArray, Charset charset) {
501512
* @return new instance
502513
*/
503514
public static Bytes from(UUID uuid) {
504-
Objects.requireNonNull(uuid);
505-
return wrap(Util.getBytesFromUUID(uuid).array());
515+
return wrap(Util.getBytesFromUUID(Objects.requireNonNull(uuid)).array());
506516
}
507517

508518
/**
@@ -606,10 +616,7 @@ public static Bytes parseBase64(String base64String) {
606616
* @return decoded instance
607617
*/
608618
public static Bytes parse(String encoded, BinaryToTextEncoding.Decoder decoder) {
609-
Objects.requireNonNull(encoded, "encoded data must not be null");
610-
Objects.requireNonNull(decoder, "passed decoder instance must no be null");
611-
612-
return wrap(decoder.decode(encoded));
619+
return wrap(Objects.requireNonNull(decoder, "passed decoder instance must no be null").decode(Objects.requireNonNull(encoded, "encoded data must not be null")));
613620
}
614621

615622
/**
@@ -763,9 +770,7 @@ public Bytes append(CharSequence stringUtf8) {
763770
* @return appended instance
764771
*/
765772
public Bytes append(CharSequence string, Charset charset) {
766-
Objects.requireNonNull(charset);
767-
Objects.requireNonNull(string);
768-
return transform(new BytesTransformer.ConcatTransformer(string.toString().getBytes(charset)));
773+
return transform(new BytesTransformer.ConcatTransformer(Objects.requireNonNull(string).toString().getBytes(Objects.requireNonNull(charset))));
769774
}
770775

771776
/**
@@ -1059,8 +1064,7 @@ public boolean validateNotOnlyZeros() {
10591064
* @return true if all validators return true
10601065
*/
10611066
public boolean validate(BytesValidator... bytesValidators) {
1062-
Objects.requireNonNull(bytesValidators);
1063-
return BytesValidators.and(bytesValidators).validate(internalArray());
1067+
return BytesValidators.and(Objects.requireNonNull(bytesValidators)).validate(internalArray());
10641068
}
10651069

10661070
/* ATTRIBUTES ************************************************************************************************/

src/main/java/at/favre/lib/bytes/MutableBytes.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -119,7 +119,7 @@ public MutableBytes secureWipe() {
119119
* @return this instance
120120
*/
121121
public MutableBytes secureWipe(SecureRandom random) {
122-
Objects.requireNonNull(random, "must be non-null random");
122+
Objects.requireNonNull(random, "random param must not be null");
123123
if (length() > 0) {
124124
random.nextBytes(internalArray());
125125
}

0 commit comments

Comments
 (0)