Skip to content

Commit b789099

Browse files
mernstvy
authored andcommitted
Fix nullability docs in StrSubstitutor (#1422)
1 parent 6b478f6 commit b789099

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

log4j-core/src/main/java/org/apache/logging/log4j/core/lookup/StrSubstitutor.java

+4-4
Original file line numberDiff line numberDiff line change
@@ -1235,13 +1235,13 @@ public StrMatcher getVariablePrefixMatcher() {
12351235
* allowing advanced prefix matches.
12361236
* </p>
12371237
*
1238-
* @param prefixMatcher the prefix matcher to use, null ignored
1238+
* @param prefixMatcher the prefix matcher to use, must not be null
12391239
* @return this, to enable chaining
12401240
* @throws IllegalArgumentException if the prefix matcher is null
12411241
*/
12421242
public StrSubstitutor setVariablePrefixMatcher(final StrMatcher prefixMatcher) {
12431243
if (prefixMatcher == null) {
1244-
throw new IllegalArgumentException("Variable prefix matcher must not be null!");
1244+
throw new IllegalArgumentException("Parameter prefixMatcher must not be null!");
12451245
}
12461246
this.prefixMatcher = prefixMatcher;
12471247
return this;
@@ -1304,13 +1304,13 @@ public StrMatcher getVariableSuffixMatcher() {
13041304
* allowing advanced suffix matches.
13051305
* </p>
13061306
*
1307-
* @param suffixMatcher the suffix matcher to use, null ignored
1307+
* @param suffixMatcher the suffix matcher to use, must not be null
13081308
* @return this, to enable chaining
13091309
* @throws IllegalArgumentException if the suffix matcher is null
13101310
*/
13111311
public StrSubstitutor setVariableSuffixMatcher(final StrMatcher suffixMatcher) {
13121312
if (suffixMatcher == null) {
1313-
throw new IllegalArgumentException("Variable suffix matcher must not be null!");
1313+
throw new IllegalArgumentException("Parameter suffixMatcher must not be null!");
13141314
}
13151315
this.suffixMatcher = suffixMatcher;
13161316
return this;

0 commit comments

Comments
 (0)