DATAJPA-1443 - Polishing.

Original pull request: spring-projects#295.

Fixed more JavaDoc warnings by completing parameter and return value description.

Author: schauder

src/main/java/org/springframework/data/jpa/domain/Specifications.java

@@ -55,9 +55,9 @@ public class Specifications<T> implements Specification<T>, Serializable {
 	 * Simple static factory method to add some syntactic sugar around a {@link Specification}.
 	 *
 	 * @deprecated since 2.0, use {@link Specification#where} instead
-	 * @param <T>
+	 * @param <T> type parameter for the specification parameter.
 	 * @param spec can be {@literal null}.
-	 * @return
+	 * @return a new Specifcations instance. Guaranteed to be not {@code null}.
 	 */
 	@Deprecated
 	public static <T> Specifications<T> where(@Nullable Specification<T> spec) {
@@ -69,7 +69,7 @@ public static <T> Specifications<T> where(@Nullable Specification<T> spec) {
 	 *
 	 * @deprecated since 2.0, use {@link Specification#and} instead
 	 * @param other can be {@literal null}.
-	 * @return
+	 * @return a new Specifications instance combining this and the parameter instance. Guaranteed to be not {@code null}.
 	 */
 	@Deprecated
 	public Specifications<T> and(@Nullable Specification<T> other) {
@@ -81,7 +81,7 @@ public Specifications<T> and(@Nullable Specification<T> other) {
 	 *
 	 * @deprecated since 2.0, use {@link Specification#or} instead
 	 * @param other can be {@literal null}.
-	 * @return
+	 * @return a new Specifications instance combining this and the parameter instance. Guaranteed to be not {@code null}.
 	 */
 	@Deprecated
 	public Specifications<T> or(@Nullable Specification<T> other) {
@@ -92,9 +92,9 @@ public Specifications<T> or(@Nullable Specification<T> other) {
 	 * Negates the given {@link Specification}.
 	 *
 	 * @deprecated since 2.0, use {@link Specification#not} instead
-	 * @param <T>
+	 * @param <T> type parameter for the specification parameter.
 	 * @param spec can be {@literal null}.
-	 * @return
+	 * @return a new Specifications instance combining this and the parameter instance. Guaranteed to be not {@code null}.
 	 */
 	@Deprecated
 	public static <T> Specifications<T> not(@Nullable Specification<T> spec) {

src/main/java/org/springframework/data/jpa/util/HibernateProxyDetector.java

@@ -19,6 +19,7 @@
 
 import org.hibernate.proxy.HibernateProxy;
 import org.springframework.data.util.ProxyUtils.ProxyDetector;
+import org.springframework.lang.Nullable;
 import org.springframework.util.ClassUtils;
 
 /**
@@ -43,7 +44,8 @@ public Class<?> getUserType(Class<?> type) {
 				.orElse(type);
 	}
 
-	private static final Class<?> loadHibernateProxyType() {
+	@Nullable
+	private static Class<?> loadHibernateProxyType() {
 
 		try {
 			return ClassUtils.forName("org.hibernate.proxy.HibernateProxy", HibernateProxyDetector.class.getClassLoader());