Make it obvious that DurationFormatUtils.Token wraps non-null values

- Add Objects.requireNonNull() check
- Javadoc
This commit is contained in:
Gary Gregory 2023-07-16 16:42:01 -04:00
parent 514f6751ab
commit 85212f1738
1 changed files with 7 additions and 7 deletions

View File

@ -21,6 +21,7 @@ import java.util.ArrayList;
import java.util.Calendar; import java.util.Calendar;
import java.util.Date; import java.util.Date;
import java.util.GregorianCalendar; import java.util.GregorianCalendar;
import java.util.Objects;
import java.util.TimeZone; import java.util.TimeZone;
import java.util.stream.Stream; import java.util.stream.Stream;
@ -585,22 +586,21 @@ public class DurationFormatUtils {
/** /**
* Wraps a token around a value. A value would be something like a 'Y'. * Wraps a token around a value. A value would be something like a 'Y'.
* *
* @param value to wrap * @param value to wrap, non-null.
*/ */
Token(final Object value) { Token(final Object value) {
this.value = value; this(value, 1);
this.count = 1;
} }
/** /**
* Wraps a token around a repeated number of a value, for example it would * Wraps a token around a repeated number of a value, for example it would
* store 'yyyy' as a value for y and a count of 4. * store 'yyyy' as a value for y and a count of 4.
* *
* @param value to wrap * @param value to wrap, non-null.
* @param count to wrap * @param count to wrap.
*/ */
Token(final Object value, final int count) { Token(final Object value, final int count) {
this.value = value; this.value = Objects.requireNonNull(value, "value");
this.count = count; this.count = count;
} }
@ -623,7 +623,7 @@ public class DurationFormatUtils {
/** /**
* Gets the particular value this token represents. * Gets the particular value this token represents.
* *
* @return Object value * @return Object value, non-null.
*/ */
Object getValue() { Object getValue() {
return value; return value;