Fix #179: Avoid double wrapping with Collections.unmodifiableList()

jakartaee · Apr 4, 2018 · 54e61d0 · 54e61d0
1 parent cb9cb44
commit 54e61d0
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 8 deletions.
diff --git a/api/client/src/main/java/javax/websocket/ClientEndpointConfig.java b/api/client/src/main/java/javax/websocket/ClientEndpointConfig.java
@@ -152,10 +152,10 @@ public static ClientEndpointConfig.Builder create() {
         */
        public ClientEndpointConfig build() {
            return new DefaultClientEndpointConfig(
-               Collections.unmodifiableList(this.preferredSubprotocols),
-               Collections.unmodifiableList(this.extensions),
-               Collections.unmodifiableList(this.encoders),
-               Collections.unmodifiableList(this.decoders),
+               this.preferredSubprotocols,
+               this.extensions,
+               this.encoders,
+               this.decoders,
                this.clientEndpointConfigurator);
        }
 

diff --git a/api/server/src/main/java/javax/websocket/server/ServerEndpointConfig.java b/api/server/src/main/java/javax/websocket/server/ServerEndpointConfig.java
@@ -278,10 +278,10 @@ public ServerEndpointConfig build() {
             return new DefaultServerEndpointConfig(
                     this.endpointClass,
                     this.path,
-                    Collections.unmodifiableList(this.subprotocols),
-                    Collections.unmodifiableList(this.extensions),
-                    Collections.unmodifiableList(this.encoders),
-                    Collections.unmodifiableList(this.decoders),
+                    this.subprotocols,
+                    this.extensions,
+                    this.encoders,
+                    this.decoders,
                     this.serverEndpointConfigurator
                  );
         }