Polishing.

Add missing Override annotations, consistently use literal tags for boolean values.

Author: mp911de

src/main/java/org/springframework/data/auditing/AuditingHandlerSupport.java

@@ -61,7 +61,7 @@ public AuditingHandlerSupport(PersistentEntities entities) {
 	/**
 	 * Setter do determine if {@link Auditable#setCreatedDate(TemporalAccessor)}} and
 	 * {@link Auditable#setLastModifiedDate(TemporalAccessor)} shall be filled with the current Java time. Defaults to
-	 * {@code true}. One might set this to {@code false} to use database features to set entity time.
+	 * {@literal true}. One might set this to {@literal false} to use database features to set entity time.
 	 *
 	 * @param dateTimeForNow the dateTimeForNow to set
 	 */
@@ -71,7 +71,7 @@ public void setDateTimeForNow(boolean dateTimeForNow) {
 
 	/**
 	 * Set this to true if you want to treat entity creation as modification and thus setting the current date as
-	 * modification date during creation, too. Defaults to {@code true}.
+	 * modification date during creation, too. Defaults to {@literal true}.
 	 *
 	 * @param modifyOnCreation if modification information shall be set on creation, too
 	 */

src/main/java/org/springframework/data/domain/Window.java

@@ -68,9 +68,9 @@ static <T> Window<T> from(List<T> items, IntFunction<? extends ScrollPosition> p
 	int size();
 
 	/**
-	 * Returns {@code true} if this window contains no elements.
+	 * Returns {@literal true} if this window contains no elements.
 	 *
-	 * @return {@code true} if this window contains no elements
+	 * @return {@literal true} if this window contains no elements
 	 */
 	@Override
 	boolean isEmpty();
@@ -102,7 +102,7 @@ default boolean isLast() {
 	 * Returns whether the underlying scroll mechanism can provide a {@link ScrollPosition} at {@code index}.
 	 *
 	 * @param index
-	 * @return {@code true} if a {@link ScrollPosition} can be created; {@code false} otherwise.
+	 * @return {@literal true} if a {@link ScrollPosition} can be created; {@literal false} otherwise.
 	 * @see #positionAt(int)
 	 */
 	default boolean hasPosition(int index) {

src/main/java/org/springframework/data/expression/ValueExpression.java

@@ -48,7 +48,7 @@ default ExpressionDependencies getExpressionDependencies() {
 	/**
 	 * Returns whether the expression is a literal expression (that doesn't actually require evaluation).
 	 *
-	 * @return {@code true} if the expression is a literal expression; {@code false} if the expression can yield a
+	 * @return {@literal true} if the expression is a literal expression; {@literal false} if the expression can yield a
 	 *         different result upon {@link #evaluate(ValueEvaluationContext) evaluation}.
 	 */
 	boolean isLiteral();

src/main/java/org/springframework/data/mapping/Alias.java

@@ -22,7 +22,7 @@
 
 /**
  * A container object which may or may not contain a type alias value. If a value is present, {@code isPresent()} will
- * return {@code true} and {@link #getValue()} will return the value.
+ * return {@literal true} and {@link #getValue()} will return the value.
  * <p>
  * Additional methods that depend on the presence or absence of a contained value are provided, such as
  * {@link #hasValue(Object)} or {@link #isPresent()}

src/main/java/org/springframework/data/mapping/PersistentEntity.java

@@ -229,7 +229,7 @@ default P getPersistentProperty(Class<? extends Annotation> annotationType) {
 	 * Returns whether the property is transient.
 	 *
 	 * @param property name of the property.
-	 * @return {@code true} if the property is transient. Applies only for existing properties. {@code false} if the
+	 * @return {@literal true} if the property is transient. Applies only for existing properties. {@literal false} if the
 	 *         property does not exist or is not transient.
 	 * @since 3.3
 	 */

src/main/java/org/springframework/data/mapping/model/AnnotationBasedPersistentProperty.java

@@ -264,10 +264,10 @@ public <A extends Annotation> A findPropertyOrOwnerAnnotation(Class<A> annotatio
 	}
 
 	/**
-	 * Returns whether the property carries the an annotation of the given type.
+	 * Returns whether the property carries the annotation of the given type.
 	 *
 	 * @param annotationType the annotation type to look up.
-	 * @return
+	 * @return {@literal true} if the annotation is present, {@literal false} otherwise.
 	 */
 	@Override
 	public boolean isAnnotationPresent(Class<? extends Annotation> annotationType) {

src/main/java/org/springframework/data/mapping/model/KotlinValueUtils.java

@@ -348,7 +348,7 @@ public Class<?> getParameterType() {
 		}
 
 		/**
-		 * @return {@code true} if the value hierarchy applies boxing.
+		 * @return {@literal true} if the value hierarchy applies boxing.
 		 */
 		public boolean appliesBoxing() {
 			return applyBoxing;

src/main/java/org/springframework/data/mapping/model/PersistentEntityParameterValueProvider.java

@@ -26,9 +26,7 @@
 
 /**
  * {@link ParameterValueProvider} based on a {@link PersistentEntity} to use a {@link PropertyValueProvider} to look up
- * the value of the property referenced by the given {@link Parameter}. Additionally, a
- * {@link DefaultSpELExpressionEvaluator} can be configured to get property value resolution trumped by a SpEL
- * expression evaluation.
+ * the value of the property referenced by the given {@link Parameter}.
  *
  * @author Oliver Gierke
  * @author Johannes Englmeier
@@ -42,18 +40,13 @@ public class PersistentEntityParameterValueProvider<P extends PersistentProperty
 	private final @Nullable Object parent;
 
 	public PersistentEntityParameterValueProvider(PersistentEntity<?, P> entity, PropertyValueProvider<P> provider,
-			Object parent) {
+			@Nullable Object parent) {
 		this.entity = entity;
 		this.provider = provider;
 		this.parent = parent;
 	}
 
 	@Override
-	@Nullable
-	private static Object getTransientDefault(Class<?> parameterType) {
-		return parameterType.isPrimitive() ? ReflectionUtils.getPrimitiveDefault(parameterType) : null;
-	}
-
 	@Nullable
 	@SuppressWarnings("unchecked")
 	public <T> T getParameterValue(Parameter<T, P> parameter) {
@@ -82,4 +75,10 @@ public <T> T getParameterValue(Parameter<T, P> parameter) {
 
 		return provider.getPropertyValue(property);
 	}
+
+	@Nullable
+	private static Object getTransientDefault(Class<?> parameterType) {
+		return parameterType.isPrimitive() ? ReflectionUtils.getPrimitiveDefault(parameterType) : null;
+	}
+
 }

src/main/java/org/springframework/data/projection/EntityProjection.java

@@ -158,14 +158,14 @@ public TypeInformation<?> getActualDomainType() {
 	}
 
 	/**
-	 * @return {@code true} if the {@link #getMappedType()} is a projection.
+	 * @return {@literal true} if the {@link #getMappedType()} is a projection.
 	 */
 	public boolean isProjection() {
 		return projection;
 	}
 
 	/**
-	 * @return {@code true} if the {@link #getMappedType()} is a closed projection.
+	 * @return {@literal true} if the {@link #getMappedType()} is a closed projection.
 	 */
 	public boolean isClosedProjection() {
 		return isProjection()

src/main/java/org/springframework/data/projection/EntityProjectionIntrospector.java

@@ -222,13 +222,13 @@ public interface ProjectionPredicate {
 		 *
 		 * @param target the target type.
 		 * @param underlyingType the underlying type.
-		 * @return {@code true} if the input argument matches the predicate, otherwise {@code false}.
+		 * @return {@literal true} if the input argument matches the predicate, otherwise {@literal false}.
 		 */
 		boolean test(Class<?> target, Class<?> underlyingType);
 
 		/**
 		 * Return a composed predicate that represents a short-circuiting logical AND of this predicate and another. When
-		 * evaluating the composed predicate, if this predicate is {@code false}, then the {@code other} predicate is not
+		 * evaluating the composed predicate, if this predicate is {@literal false}, then the {@code other} predicate is not
 		 * evaluated.
 		 * <p>
 		 * Any exceptions thrown during evaluation of either predicate are relayed to the caller; if evaluation of this