@@ -104,13 +104,13 @@ public static HashParameters concat(Parameters... parameters) {
104104 * @return a concatenation, never {@code null}.
105105 */
106106 public static HashParameters concat (Iterable <Parameters > parameters ) {
107- ArrayList <Map <String , List <String >>> prms = new ArrayList <>();
107+ ArrayList <Map <String , List <String >>> _params = new ArrayList <>();
108108 for (Parameters p : parameters ) {
109109 if (p != null ) {
110- prms .add (p .toMap ());
110+ _params .add (p .toMap ());
111111 }
112112 }
113- return concat (prms );
113+ return concat (_params );
114114 }
115115
116116 private static HashParameters concat (List <Map <String , List <String >>> prms ) {
@@ -240,9 +240,9 @@ public List<String> computeSingleIfAbsent(String key, Function<String, String> v
240240 }
241241
242242 @ Override
243- public void putAll (Parameters parameters ) {
243+ public Parameters putAll (Parameters parameters ) {
244244 if (parameters == null ) {
245- return ;
245+ return null ;
246246 }
247247
248248 for (Map .Entry <String , List <String >> entry : parameters .toMap ().entrySet ()) {
@@ -251,58 +251,62 @@ public void putAll(Parameters parameters) {
251251 content .put (entry .getKey (), Collections .unmodifiableList (values ));
252252 }
253253 }
254+ return parameters ;
254255 }
255256
256257 @ Override
257- public void add (String key , String ... values ) {
258+ public Parameters add (String key , String ... values ) {
258259 Objects .requireNonNull (key , "Parameter 'key' is null!" );
259260 if (values == null || values .length == 0 ) {
260261 // do not necessarily create an entry in the map, simply immediately return
261- return ;
262+ return null ;
262263 }
263264
264265 content .compute (key , (s , list ) -> {
265266 if (list == null ) {
266- return Collections . unmodifiableList ( new ArrayList <>( Arrays . asList ( values )) );
267+ return List . of ( values );
267268 } else {
268269 ArrayList <String > newValues = new ArrayList <>(list .size () + values .length );
269270 newValues .addAll (list );
270271 newValues .addAll (Arrays .asList (values ));
271272 return Collections .unmodifiableList (newValues );
272273 }
273274 });
275+ return null ;
274276 }
275277
276278 @ Override
277- public void add (String key , Iterable <String > values ) {
279+ public Parameters add (String key , Iterable <String > values ) {
278280 Objects .requireNonNull (key , "Parameter 'key' is null!" );
279281 List <String > vls = internalListCopy (values );
280282 if (vls == null ) {
281283 // do not necessarily create an entry in the map, simply immediately return
282- return ;
284+ return null ;
283285 }
284286
285287 content .compute (key , (s , list ) -> {
286288 if (list == null ) {
287- return Collections . unmodifiableList ( vls ) ;
289+ return vls ;
288290 } else {
289291 ArrayList <String > newValues = new ArrayList <>(list .size () + vls .size ());
290292 newValues .addAll (list );
291293 newValues .addAll (vls );
292294 return Collections .unmodifiableList (newValues );
293295 }
294296 });
297+ return null ;
295298 }
296299
297300 @ Override
298- public void addAll (Parameters parameters ) {
301+ public Parameters addAll (Parameters parameters ) {
299302 if (parameters == null ) {
300- return ;
303+ return null ;
301304 }
302305 Map <String , List <String >> map = parameters .toMap ();
303306 for (Map .Entry <String , List <String >> entry : map .entrySet ()) {
304307 add (entry .getKey (), entry .getValue ());
305308 }
309+ return parameters ;
306310 }
307311
308312 @ Override
0 commit comments