Polishing
This commit is contained in:
@@ -283,12 +283,14 @@ public class DataBinder implements PropertyEditorRegistry, TypeConverter {
|
||||
protected AbstractPropertyBindingResult createDirectFieldBindingResult() {
|
||||
DirectFieldBindingResult result = new DirectFieldBindingResult(getTarget(),
|
||||
getObjectName(), isAutoGrowNestedPaths());
|
||||
|
||||
if (this.conversionService != null) {
|
||||
result.initConversion(this.conversionService);
|
||||
}
|
||||
if (this.messageCodesResolver != null) {
|
||||
result.setMessageCodesResolver(this.messageCodesResolver);
|
||||
}
|
||||
|
||||
return result;
|
||||
}
|
||||
|
||||
|
||||
@@ -19,9 +19,9 @@ package org.springframework.web.socket.server.jetty;
|
||||
import java.io.IOException;
|
||||
import java.security.Principal;
|
||||
import java.util.ArrayList;
|
||||
import java.util.LinkedList;
|
||||
import java.util.List;
|
||||
import java.util.Map;
|
||||
import java.util.Set;
|
||||
import javax.servlet.ServletContext;
|
||||
import javax.servlet.http.HttpServletRequest;
|
||||
import javax.servlet.http.HttpServletResponse;
|
||||
@@ -162,8 +162,9 @@ public class JettyRequestUpgradeStrategy implements RequestUpgradeStrategy, Serv
|
||||
}
|
||||
|
||||
private List<WebSocketExtension> buildWebSocketExtensions() {
|
||||
List<WebSocketExtension> result = new ArrayList<>();
|
||||
for (String name : this.factoryAdapter.getFactory().getExtensionFactory().getExtensionNames()) {
|
||||
Set<String> names = this.factoryAdapter.getFactory().getExtensionFactory().getExtensionNames();
|
||||
List<WebSocketExtension> result = new ArrayList<>(names.size());
|
||||
for (String name : names) {
|
||||
result.add(new WebSocketExtension(name));
|
||||
}
|
||||
return result;
|
||||
@@ -217,10 +218,10 @@ public class JettyRequestUpgradeStrategy implements RequestUpgradeStrategy, Serv
|
||||
this.handler = handler;
|
||||
this.selectedProtocol = protocol;
|
||||
if (CollectionUtils.isEmpty(extensions)) {
|
||||
this.extensionConfigs = new LinkedList<>();
|
||||
this.extensionConfigs = new ArrayList<>(0);
|
||||
}
|
||||
else {
|
||||
this.extensionConfigs = new ArrayList<>();
|
||||
this.extensionConfigs = new ArrayList<>(extensions.size());
|
||||
for (WebSocketExtension extension : extensions) {
|
||||
this.extensionConfigs.add(new WebSocketToJettyExtensionConfigAdapter(extension));
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user