Skip to content
Draft
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@

<groupId>org.springframework.data</groupId>
<artifactId>spring-data-commons</artifactId>
<version>4.0.2-SNAPSHOT</version>
<version>4.0.x-GH-3431-SNAPSHOT</version>

<name>Spring Data Core</name>
<description>Core Spring concepts underpinning every Spring Data module.</description>
Expand Down Expand Up @@ -34,6 +34,7 @@
<xmlbeam>1.4.26</xmlbeam>
<java-module-name>spring.data.commons</java-module-name>
<kotlin.api.target>1.8</kotlin.api.target>
<spring>7.0.3-SNAPSHOT</spring>
</properties>

<dependencies>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -51,8 +51,6 @@
*/
class InstanceCreatorMetadataDiscoverer {

private static final ParameterNameDiscoverer PARAMETER_NAME_DISCOVERER = new DefaultParameterNameDiscoverer();

/**
* Discover an entity creator
*
Expand Down Expand Up @@ -146,7 +144,7 @@ private static <T, P extends PersistentProperty<P>> FactoryMethod<Object, P> get
Annotation[][] parameterAnnotations = method.getParameterAnnotations();
List<TypeInformation<?>> types = entity.getTypeInformation().getParameterTypes(method);

String[] parameterNames = PARAMETER_NAME_DISCOVERER.getParameterNames(method);
String[] parameterNames = DefaultParameterNameDiscoverer.getSharedInstance().getParameterNames(method);

for (int i = 0; i < parameters.length; i++) {

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -213,8 +213,6 @@ <T, P extends PersistentProperty<P>> PreferredConstructor<T, P> discover(TypeInf
}
};

private static final ParameterNameDiscoverer PARAMETER_NAME_DISCOVERER = new DefaultParameterNameDiscoverer();

/**
* Find the appropriate discoverer for {@code type}.
*
Expand Down Expand Up @@ -249,7 +247,7 @@ private static <T, P extends PersistentProperty<P>> PreferredConstructor<T, P> b
}

List<TypeInformation<?>> parameterTypes = typeInformation.getParameterTypes(constructor);
String[] parameterNames = PARAMETER_NAME_DISCOVERER.getParameterNames(constructor);
String[] parameterNames = DefaultParameterNameDiscoverer.getSharedInstance().getParameterNames(constructor);

Parameter<Object, P>[] parameters = new Parameter[parameterTypes.size()];
Annotation[][] parameterAnnotations = constructor.getParameterAnnotations();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -102,7 +102,7 @@ public CodeBlock decorate() {
for (int i = 0; i < it.getParameterCount(); i++) {

MethodParameter parameter = new MethodParameter(it, i);
parameter.initParameterNameDiscovery(new DefaultParameterNameDiscoverer());
parameter.initParameterNameDiscovery(DefaultParameterNameDiscoverer.getSharedInstance());

callbackMethod.addParameter(parameter.getParameterType(), parameter.getParameterName());
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -63,9 +63,8 @@ class MethodMetadata {
Map<String, MethodParameter> methodParameters = new LinkedHashMap<>();

ResolvableType repositoryInterface = ResolvableType.forClass(repositoryInformation.getRepositoryInterface());
ParameterNameDiscoverer nameDiscoverer = new DefaultParameterNameDiscoverer();

initializeMethodArguments(method, nameDiscoverer, repositoryInterface, methodArguments, methodParameters);
initializeMethodArguments(method, DefaultParameterNameDiscoverer.getSharedInstance(), repositoryInterface, methodArguments, methodParameters);

this.methodArguments = Collections.unmodifiableMap(methodArguments);
this.methodParameters = Collections.unmodifiableMap(methodParameters);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,9 +23,7 @@
import java.util.List;
import java.util.function.Function;

import org.springframework.core.DefaultParameterNameDiscoverer;
import org.springframework.core.MethodParameter;
import org.springframework.core.ParameterNameDiscoverer;
import org.springframework.core.ResolvableType;
import org.springframework.data.domain.Limit;
import org.springframework.data.domain.Pageable;
Expand Down Expand Up @@ -56,8 +54,6 @@ public abstract class Parameters<S extends Parameters<S, T>, T extends Parameter
"Either use @%s on all parameters except %s and %s typed once, or none at all", Param.class.getSimpleName(),
Pageable.class.getSimpleName(), Sort.class.getSimpleName());

private static final ParameterNameDiscoverer PARAMETER_NAME_DISCOVERER = new DefaultParameterNameDiscoverer();

private final int vectorIndex;
private final int scoreIndex;
private final int scoreRangeIndex;
Expand Down Expand Up @@ -102,8 +98,6 @@ protected Parameters(ParametersSource parametersSource, Function<MethodParameter
MethodParameter methodParameter = new MethodParameter(method, i)
.withContainingClass(parametersSource.getContainingClass());

methodParameter.initParameterNameDiscovery(PARAMETER_NAME_DISCOVERER);

T parameter = parameterFactory.apply(methodParameter);

if (parameter.isSpecialParameter() && parameter.isNamedParameter()) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,6 @@
*/
class MethodParameters {

private final static ParameterNameDiscoverer discoverer = new DefaultParameterNameDiscoverer();
private final List<MethodParameter> parameters;

/**
Expand All @@ -66,7 +65,7 @@ public MethodParameters(Method method, Optional<AnnotationAttribute> namingAnnot
for (int i = 0; i < method.getParameterCount(); i++) {

MethodParameter parameter = new AnnotationNamingMethodParameter(method, i, namingAnnotation);
parameter.initParameterNameDiscovery(discoverer);
parameter.initParameterNameDiscovery(DefaultParameterNameDiscoverer.getSharedInstance());
parameters.add(parameter);
}
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,6 @@
*/
public class NullnessMethodInvocationValidator implements MethodInterceptor {

private final ParameterNameDiscoverer discoverer = new DefaultParameterNameDiscoverer();
private final Map<Method, MethodNullness> nullabilityCache = new ConcurrentHashMap<>(16);

/**
Expand All @@ -79,7 +78,7 @@ public static boolean supports(Class<?> type) {

if (nullness == null) {

nullness = MethodNullness.of(method, discoverer);
nullness = MethodNullness.of(method, DefaultParameterNameDiscoverer.getSharedInstance());
nullabilityCache.put(method, nullness);
}

Expand Down
Loading