Navigation Menu

Skip to content

Commit

Permalink
Use synchronizedList which accepts null values
Browse files Browse the repository at this point in the history
  • Loading branch information
jonmv committed Nov 13, 2020
1 parent b4ae98c commit 897a048
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions jdisc_core/src/main/java/com/yahoo/jdisc/HeaderFields.java
Expand Up @@ -5,6 +5,7 @@

import java.util.ArrayList;
import java.util.Collection;
import java.util.Collections;
import java.util.List;
import java.util.Map;
import java.util.Set;
Expand Down Expand Up @@ -140,14 +141,14 @@ public void addAll(Map<? extends String, ? extends List<String>> values) {
* <code>key</code>.
*/
public List<String> put(String key, String value) {
ArrayList<String> list = new ArrayList<String>(1);
List<String> list = Collections.synchronizedList(new ArrayList<>(1));
list.add(value);
return content.put(key, list);
}

@Override
public List<String> put(String key, List<String> value) {
return content.put(key, new ArrayList<>(value));
return content.put(key, Collections.synchronizedList(new ArrayList<>(value)));
}

@Override
Expand Down

0 comments on commit 897a048

Please sign in to comment.