From 4b0de4ce4bd65ac3be1867e5671b9ab9ac524669 Mon Sep 17 00:00:00 2001 From: Jon Marius Venstad Date: Fri, 13 Nov 2020 12:44:36 +0100 Subject: Use a thread safe class for holding header values --- .../src/main/java/com/yahoo/jdisc/HeaderFields.java | 15 +++++++-------- 1 file changed, 7 insertions(+), 8 deletions(-) (limited to 'jdisc_core/src/main/java/com/yahoo/jdisc') diff --git a/jdisc_core/src/main/java/com/yahoo/jdisc/HeaderFields.java b/jdisc_core/src/main/java/com/yahoo/jdisc/HeaderFields.java index 5c35bed2cec..2f7cd59c737 100644 --- a/jdisc_core/src/main/java/com/yahoo/jdisc/HeaderFields.java +++ b/jdisc_core/src/main/java/com/yahoo/jdisc/HeaderFields.java @@ -3,7 +3,12 @@ package com.yahoo.jdisc; import com.google.common.collect.ImmutableList; -import java.util.*; +import java.util.ArrayList; +import java.util.Collection; +import java.util.List; +import java.util.Map; +import java.util.Set; +import java.util.concurrent.ConcurrentSkipListMap; /** * This is an encapsulation of the header fields that belong to either a {@link Request} or a {@link Response}. It is @@ -14,13 +19,7 @@ import java.util.*; */ public class HeaderFields implements Map> { - private final TreeMap> content = new TreeMap<>(new Comparator() { - - @Override - public int compare(String lhs, String rhs) { - return lhs.compareToIgnoreCase(rhs); - } - }); + private final ConcurrentSkipListMap> content = new ConcurrentSkipListMap<>(String::compareToIgnoreCase); @Override public int size() { -- cgit v1.2.3