You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Having @deprecated in the javadoc without @Deprecated on the element can lead to an inconsistent view of whether or not something is deprecated. The JLS requires compilers to warn when someone uses code that is annotated with @Deprecated. It does not require them to warn when code is documented with @deprecated, but javac does so. It would be good to always use @deprecated and @Deprecated together so that our code avoids this potential inconsistency.
If we do this, it would also make sense to change this setting in Eclipse:
https://checkstyle.sourceforge.io/checks/annotation/missingdeprecated.html
Having
@deprecated
in the javadoc without@Deprecated
on the element can lead to an inconsistent view of whether or not something is deprecated. The JLS requires compilers to warn when someone uses code that is annotated with@Deprecated
. It does not require them to warn when code is documented with@deprecated
, butjavac
does so. It would be good to always use@deprecated
and@Deprecated
together so that our code avoids this potential inconsistency.If we do this, it would also make sense to change this setting in Eclipse:
spring-javaformat/spring-javaformat-eclipse/io.spring.javaformat.eclipse/src/io/spring/javaformat/eclipse/projectsettings/org.eclipse.jdt.core.prefs
Line 48 in cea1d86
This would also align with the cleanup settings where
@Deprecated
is added automatically when the javadoc contains@deprecated
:spring-javaformat/spring-javaformat-eclipse/io.spring.javaformat.eclipse/src/io/spring/javaformat/eclipse/projectsettings/org.eclipse.jdt.ui.prefs
Line 4 in cea1d86
The text was updated successfully, but these errors were encountered: