HHH-13925 Fix malformed @author tags in javadoc

You're not supposed to use unescaped "<" or ">".

This wasn't detected by checkstyle in Gradle 5, but it is now, and that
fails the build.

hibernate-core/src/main/java/org/hibernate/cfg/JPAIndexHolder.java

@@ -13,7 +13,7 @@
 import javax.persistence.Index;
 
 /**
- * @author Strong Liu <stliu@hibernate.org>
+ * @author <a href="mailto:stliu@hibernate.org">Strong Liu</a>
  */
 public class JPAIndexHolder {
 	private final String name;

hibernate-core/src/main/java/org/hibernate/cfg/annotations/QueryHintDefinition.java

@@ -24,7 +24,7 @@
 import org.hibernate.internal.util.LockModeConverter;
 
 /**
- * @author Strong Liu <stliu@hibernate.org>
+ * @author <a href="mailto:stliu@hibernate.org">Strong Liu</a>
  */
 public class QueryHintDefinition {
 	private final Map<String, Object> hintsMap;

hibernate-core/src/main/java/org/hibernate/dialect/AbstractHANADialect.java

@@ -117,8 +117,8 @@
  * <p>
  * Note: This dialect is configured to create foreign keys with {@code on update cascade}.
  *
- * @author Andrew Clemons <andrew.clemons@sap.com>
- * @author Jonathan Bregler <jonathan.bregler@sap.com>
+ * @author <a href="mailto:andrew.clemons@sap.com">Andrew Clemons</a>
+ * @author <a href="mailto:jonathan.bregler@sap.com">Jonathan Bregler</a>
  */
 public abstract class AbstractHANADialect extends Dialect {
 

hibernate-core/src/main/java/org/hibernate/dialect/FrontBaseDialect.java

@@ -32,7 +32,7 @@
  * 2.3.1, you can work-around this problem by setting the following in your
  * <tt>hibernate.properties</tt> file: <tt>hibernate.jdbc.batch_size=15</tt>
  *
- * @author Ron Lussier <tt>rlussier@lenscraft.com</tt>
+ * @author <a href="mailto:rlussier@lenscraft.com</tt">Ron Lussier</a>
  */
 public class FrontBaseDialect extends Dialect {
 

hibernate-core/src/main/java/org/hibernate/dialect/HANAColumnStoreDialect.java

@@ -26,8 +26,8 @@
  * <p>
  * Column tables are created by this dialect when using the auto-ddl feature.
  * 
- * @author Andrew Clemons <andrew.clemons@sap.com>
- * @author Jonathan Bregler <jonathan.bregler@sap.com>
+ * @author <a href="mailto:andrew.clemons@sap.com">Andrew Clemons</a>
+ * @author <a href="mailto:jonathan.bregler@sap.com">Jonathan Bregler</a>
  */
 public class HANAColumnStoreDialect extends AbstractHANADialect {
 

hibernate-core/src/main/java/org/hibernate/dialect/HANARowStoreDialect.java

@@ -22,8 +22,8 @@
  * <p>
  * Row tables are created by this dialect when using the auto-ddl feature.
  *
- * @author Andrew Clemons <andrew.clemons@sap.com>
- * @author Jonathan Bregler <jonathan.bregler@sap.com>
+ * @author <a href="mailto:andrew.clemons@sap.com">Andrew Clemons</a>
+ * @author <a href="mailto:jonathan.bregler@sap.com">Jonathan Bregler</a>
  */
 public class HANARowStoreDialect extends AbstractHANADialect {
 

hibernate-core/src/main/java/org/hibernate/dialect/MimerSQLDialect.java

@@ -20,7 +20,7 @@
  * An Hibernate 3 SQL dialect for Mimer SQL. This dialect requires Mimer SQL 9.2.1 or later
  * because of the mappings to NCLOB, BINARY, and BINARY VARYING.
  *
- * @author Fredrik lund <fredrik.alund@mimer.se>
+ * @author <a href="mailto:fredrik.alund@mimer.se">Fredrik lund</a>
  */
 @SuppressWarnings("deprecation")
 public class MimerSQLDialect extends Dialect {

hibernate-core/src/main/java/org/hibernate/engine/internal/AbstractEntityEntry.java

@@ -38,9 +38,9 @@
  * A base implementation of EntityEntry
  *
  * @author Gavin King
- * @author Emmanuel Bernard <emmanuel@hibernate.org>
+ * @author <a href="mailto:emmanuel@hibernate.org">Emmanuel Bernard</a>
  * @author Gunnar Morling
- * @author Sanne Grinovero  <sanne@hibernate.org>
+ * @author <a href="mailto:sanne@hibernate.org">Sanne Grinovero </a>
  */
 public abstract class AbstractEntityEntry implements Serializable, EntityEntry {
 	protected final Serializable id;

hibernate-core/src/main/java/org/hibernate/engine/internal/EntityEntryExtraStateHolder.java

@@ -11,7 +11,7 @@
 /**
  * Contains optional state from {@link org.hibernate.engine.spi.EntityEntry}.
  *
- * @author Emmanuel Bernard <emmanuel@hibernate.org>
+ * @author <a href="mailto:emmanuel@hibernate.org">Emmanuel Bernard</a>
  */
 public class EntityEntryExtraStateHolder implements EntityEntryExtraState {
 	private EntityEntryExtraState next;

hibernate-core/src/main/java/org/hibernate/engine/internal/ImmutableEntityEntry.java

@@ -25,9 +25,9 @@
  * immutable in terms of its internal state; the term immutable here refers to the entity it describes.
  *
  * @author Gavin King
- * @author Emmanuel Bernard <emmanuel@hibernate.org>
+ * @author <a href="mailto:emmanuel@hibernate.org">Emmanuel Bernard</a>
  * @author Gunnar Morling
- * @author Sanne Grinovero  <sanne@hibernate.org>
+ * @author <a href="mailto:sanne@hibernate.org">Sanne Grinovero </a>
  *
  * @see org.hibernate.annotations.Immutable
  */

hibernate-core/src/main/java/org/hibernate/engine/internal/MutableEntityEntry.java

@@ -22,9 +22,9 @@
  * An EntityEntry implementation for mutable entities.
  *
  * @author Gavin King
- * @author Emmanuel Bernard <emmanuel@hibernate.org>
+ * @author <a href="mailto:emmanuel@hibernate.org">Emmanuel Bernard</a>
  * @author Gunnar Morling
- * @author Sanne Grinovero  <sanne@hibernate.org>
+ * @author <a href="mailto:sanne@hibernate.org">Sanne Grinovero </a>
  */
 public final class MutableEntityEntry extends AbstractEntityEntry {
 	/**

hibernate-core/src/main/java/org/hibernate/engine/spi/EntityEntry.java

@@ -21,9 +21,9 @@
  * therefore we need to take care of its impact on memory consumption.
  *
  * @author Gavin King
- * @author Emmanuel Bernard <emmanuel@hibernate.org>
+ * @author <a href="mailto:emmanuel@hibernate.org">Emmanuel Bernard</a>
  * @author Gunnar Morling
- * @author Sanne Grinovero  <sanne@hibernate.org>
+ * @author <a href="mailto:sanne@hibernate.org">Sanne Grinovero </a>
  */
 public interface EntityEntry {
 	LockMode getLockMode();

hibernate-core/src/main/java/org/hibernate/engine/spi/EntityEntryExtraState.java

@@ -10,7 +10,7 @@
 /**
  * Navigation methods for extra state objects attached to {@link org.hibernate.engine.spi.EntityEntry}.
  *
- * @author Emmanuel Bernard <emmanuel@hibernate.org>
+ * @author <a href="mailto:emmanuel@hibernate.org">Emmanuel Bernard</a>
  */
 public interface EntityEntryExtraState {
 

hibernate-core/src/main/java/org/hibernate/engine/spi/SessionDelegatorBaseImpl.java

@@ -77,7 +77,7 @@
  * API so that only some methods need to be overridden
  * (Used by Hibernate Search).
  *
- * @author Sanne Grinovero <sanne@hibernate.org> (C) 2012 Red Hat Inc.
+ * @author <a href="mailto:sanne@hibernate.org">Sanne Grinovero</a> (C) 2012 Red Hat Inc.
  */
 @SuppressWarnings("deprecation")
 public class SessionDelegatorBaseImpl implements SessionImplementor {

hibernate-core/src/main/java/org/hibernate/graph/AttributeNode.java

@@ -13,7 +13,7 @@
 /**
  * Hibernate extension to the JPA entity-graph AttributeNode contract.
  *
- * @author Strong Liu <stliu@hibernate.org>
+ * @author <a href="mailto:stliu@hibernate.org">Strong Liu</a>
  * @author Steve Ebersole
  * @author Andrea Boriero
  */

hibernate-core/src/main/java/org/hibernate/graph/Graph.java

@@ -15,7 +15,7 @@
  *
  * Acts as a "bridge" between JPA's {@link javax.persistence.EntityGraph} and {@link javax.persistence.Subgraph}
  *
- * @author Strong Liu <stliu@hibernate.org>
+ * @author <a href="mailto:stliu@hibernate.org">Strong Liu</a>
  * @author Steve Ebersole
  * @author Andrea Boriero
  */

hibernate-core/src/main/java/org/hibernate/graph/spi/AttributeNodeImplementor.java

@@ -18,7 +18,7 @@
 /**
  * Integration version of the AttributeNode contract
  *
- * @author Strong Liu <stliu@hibernate.org>
+ * @author <a href="mailto:stliu@hibernate.org">Strong Liu</a>
  * @author Steve Ebersole
  */
 public interface AttributeNodeImplementor<J> extends AttributeNode<J>, GraphNodeImplementor<J> {

hibernate-core/src/main/java/org/hibernate/graph/spi/GraphImplementor.java

@@ -22,7 +22,7 @@
 /**
  * Integration version of the Graph contract
  *
- * @author Strong Liu <stliu@hibernate.org>
+ * @author <a href="mailto:stliu@hibernate.org">Strong Liu</a>
  * @author Steve Ebersole
  * @author Andrea Boriero
  */

hibernate-core/src/main/java/org/hibernate/graph/spi/GraphNodeImplementor.java

@@ -12,7 +12,7 @@
  * Integration version of the GraphNode contract
  *
  * @author Steve Ebersole
- * @author Strong Liu <stliu@hibernate.org>
+ * @author <a href="mailto:stliu@hibernate.org">Strong Liu</a>
  */
 public interface GraphNodeImplementor<J> extends GraphNode<J> {
 	@Override

hibernate-core/src/main/java/org/hibernate/id/factory/internal/MutableIdentifierGeneratorFactoryInitiator.java

@@ -13,7 +13,7 @@
 import org.hibernate.service.spi.ServiceRegistryImplementor;
 
 /**
- * @author Emmanuel Bernard <emmanuel@hibernate.org>
+ * @author <a href="mailto:emmanuel@hibernate.org">Emmanuel Bernard</a>
  */
 public class MutableIdentifierGeneratorFactoryInitiator implements StandardServiceInitiator<MutableIdentifierGeneratorFactory> {
 	public static final MutableIdentifierGeneratorFactoryInitiator INSTANCE = new MutableIdentifierGeneratorFactoryInitiator();

hibernate-core/src/main/java/org/hibernate/id/factory/spi/MutableIdentifierGeneratorFactory.java

@@ -12,7 +12,7 @@
 /**
  * Let people register strategies
  *
- * @author Emmanuel Bernard <emmanuel@hibernate.org>
+ * @author <a href="mailto:emmanuel@hibernate.org">Emmanuel Bernard</a>
  */
 public interface MutableIdentifierGeneratorFactory extends IdentifierGeneratorFactory, Service {
 	public void register(String strategy, Class generatorClass);

hibernate-core/src/main/java/org/hibernate/jpa/spi/HibernateEntityManagerFactoryAware.java

@@ -11,7 +11,7 @@
 /**
  * Internal contact for things that have {@link HibernateEntityManagerFactory} access.
  *
- * @author Strong Liu <stliu@hibernate.org>
+ * @author <a href="mailto:stliu@hibernate.org">Strong Liu</a>
  *
  * @deprecated (since 5.2) Why do we need an over-arching access to HibernateEntityManagerFactory across
  * multiple contract hierarchies?

hibernate-core/src/main/java/org/hibernate/jpa/spi/IdentifierGeneratorStrategyProvider.java

@@ -11,7 +11,7 @@
 /**
  * Provide a set of IdentifierGenerator strategies allowing to override the Hibernate Core default ones
  *
- * @author Emmanuel Bernard <emmanuel@hibernate.org>
+ * @author <a href="mailto:emmanuel@hibernate.org">Emmanuel Bernard</a>
  */
 public interface IdentifierGeneratorStrategyProvider {
 	/**

hibernate-core/src/main/java/org/hibernate/loader/plan/build/internal/AbstractEntityGraphVisitationStrategy.java

@@ -47,7 +47,7 @@
  * and we try to match the node to entity graph ( and subgraph ), if there is a match, then the attribute is fetched,
  * it is not, then depends on which property is used to apply this entity graph.
  *
- * @author Strong Liu <stliu@hibernate.org>
+ * @author <a href="mailto:stliu@hibernate.org">Strong Liu</a>
  * @author Brett Meyer
  */
 public abstract class AbstractEntityGraphVisitationStrategy

hibernate-core/src/main/java/org/hibernate/loader/plan/build/internal/FetchGraphLoadPlanBuildingStrategy.java

@@ -17,7 +17,7 @@
 /**
  * Loadplan building strategy for {@link javax.persistence.EntityGraph} is applied in {@code javax.persistence.fetchgraph} mode.
  *
- * @author Strong Liu <stliu@hibernate.org>
+ * @author <a href="mailto:stliu@hibernate.org">Strong Liu</a>
  */
 public class FetchGraphLoadPlanBuildingStrategy extends AbstractEntityGraphVisitationStrategy {
 	private final RootGraphImplementor rootEntityGraph;

hibernate-core/src/main/java/org/hibernate/loader/plan/build/internal/LoadGraphLoadPlanBuildingStrategy.java

@@ -20,7 +20,7 @@
 /**
  * Loadplan building strategy for {@link javax.persistence.EntityGraph} is applied in {@code javax.persistence.loadgraph} mode.
  *
- * @author Strong Liu <stliu@hibernate.org>
+ * @author <a href="mailto:stliu@hibernate.org">Strong Liu</a>
  */
 public class LoadGraphLoadPlanBuildingStrategy extends AbstractEntityGraphVisitationStrategy {
 	private final RootGraphImplementor<?> rootEntityGraph;

hibernate-core/src/main/java/org/hibernate/persister/spi/PersisterClassResolver.java

@@ -21,7 +21,7 @@
  *     <li>the default provider as chosen by Hibernate Core (best choice most of the time)</li>
  * </ol>
  *
- * @author Emmanuel Bernard <emmanuel@hibernate.org>
+ * @author <a href="mailto:emmanuel@hibernate.org">Emmanuel Bernard</a>
  * @author Steve Ebersole
  */
 public interface PersisterClassResolver extends Service {

hibernate-core/src/test/java/org/hibernate/jpa/test/ejb3configuration/Bell.java

@@ -10,7 +10,7 @@
 import javax.persistence.Id;
 
 /**
- * @author Emmanuel Bernard <emmanuel@hibernate.org>
+ * @author <a href="mailto:emmanuel@hibernate.org">Emmanuel Bernard</a>
  */
 @Entity
 public class Bell {

hibernate-core/src/test/java/org/hibernate/jpa/test/ejb3configuration/PersisterClassProviderTest.java

@@ -59,7 +59,7 @@
 import org.junit.Test;
 
 /**
- * @author Emmanuel Bernard <emmanuel@hibernate.org>
+ * @author <a href="mailto:emmanuel@hibernate.org">Emmanuel Bernard</a>
  */
 public class PersisterClassProviderTest {
 	@Test

hibernate-core/src/test/java/org/hibernate/jpa/test/ejb3configuration/SessionFactoryObserverTest.java

@@ -21,7 +21,7 @@
 import org.hibernate.jpa.boot.spi.EntityManagerFactoryBuilder;
 
 /**
- * @author Emmanuel Bernard <emmanuel@hibernate.org>
+ * @author <a href="mailto:emmanuel@hibernate.org">Emmanuel Bernard</a>
  */
 public class SessionFactoryObserverTest {
     @Test

hibernate-core/src/test/java/org/hibernate/jpa/test/ejb3configuration/id/Cable.java

@@ -13,7 +13,7 @@
 import org.hibernate.annotations.GenericGenerator;
 
 /**
- * @author Emmanuel Bernard <emmanuel@hibernate.org>
+ * @author <a href="mailto:emmanuel@hibernate.org">Emmanuel Bernard</a>
  */
 @Entity
 public class Cable {

hibernate-core/src/test/java/org/hibernate/jpa/test/ejb3configuration/id/FunkyException.java

@@ -7,7 +7,7 @@
 package org.hibernate.jpa.test.ejb3configuration.id;
 
 /**
- * @author Emmanuel Bernard <emmanuel@hibernate.org>
+ * @author <a href="mailto:emmanuel@hibernate.org">Emmanuel Bernard</a>
  */
 public class FunkyException extends RuntimeException {
 }

hibernate-core/src/test/java/org/hibernate/jpa/test/ejb3configuration/id/FunkyIdGenerator.java

@@ -13,7 +13,7 @@
 import org.hibernate.id.IdentifierGenerator;
 
 /**
- * @author Emmanuel Bernard <emmanuel@hibernate.org>
+ * @author <a href="mailto:emmanuel@hibernate.org">Emmanuel Bernard</a>
  */
 public class FunkyIdGenerator implements IdentifierGenerator {
 	public Serializable generate(SharedSessionContractImplementor session, Object object) throws HibernateException {

hibernate-core/src/test/java/org/hibernate/jpa/test/ejb3configuration/id/FunkyIdentifierGeneratorProvider.java

@@ -10,7 +10,7 @@
 import java.util.Map;
 
 /**
- * @author Emmanuel Bernard <emmanuel@hibernate.org>
+ * @author <a href="mailto:emmanuel@hibernate.org">Emmanuel Bernard</a>
  */
 public class FunkyIdentifierGeneratorProvider implements org.hibernate.jpa.spi.IdentifierGeneratorStrategyProvider {
 	public Map<String, Class<?>> getStrategies() {

hibernate-core/src/test/java/org/hibernate/jpa/test/ejb3configuration/id/IdentifierGeneratorStrategyProviderTest.java

@@ -21,7 +21,7 @@
 import org.junit.Test;
 
 /**
- * @author Emmanuel Bernard <emmanuel@hibernate.org>
+ * @author <a href="mailto:emmanuel@hibernate.org">Emmanuel Bernard</a>
  */
 public class IdentifierGeneratorStrategyProviderTest {
 	@Test

hibernate-core/src/test/java/org/hibernate/jpa/test/graphs/EntityGraphLoadPlanBuilderTest.java

@@ -43,7 +43,7 @@
 import org.hibernate.persister.entity.EntityPersister;
 
 /**
- * @author Strong Liu <stliu@hibernate.org>
+ * @author <a href="mailto:stliu@hibernate.org">Strong Liu</a>
  */
 public class EntityGraphLoadPlanBuilderTest extends BaseEntityManagerFunctionalTestCase {
 	@Override

hibernate-core/src/test/java/org/hibernate/jpa/test/graphs/named/basic/BasicAnnNamedEntityGraphTest.java

@@ -7,7 +7,7 @@
 package org.hibernate.jpa.test.graphs.named.basic;
 
 /**
- * @author Strong Liu <stliu@hibernate.org>
+ * @author <a href="mailto:stliu@hibernate.org">Strong Liu</a>
  */
 public class BasicAnnNamedEntityGraphTest extends AbstractNamedEntityGraphTest{
 	@Override

hibernate-core/src/test/java/org/hibernate/jpa/test/graphs/named/basic/BasicOrmNamedEntityGraphTest.java

@@ -7,7 +7,7 @@
 package org.hibernate.jpa.test.graphs.named.basic;
 
 /**
- * @author Strong Liu <stliu@hibernate.org>
+ * @author <a href="mailto:stliu@hibernate.org">Strong Liu</a>
  */
 public class BasicOrmNamedEntityGraphTest extends AbstractNamedEntityGraphTest{
 	@Override

hibernate-core/src/test/java/org/hibernate/jpa/test/lock/LockTimeoutPropertyTest.java

@@ -25,7 +25,7 @@
 /**
  * no need to run this on DB matrix
  *
- * @author Strong Liu <stliu@hibernate.org>
+ * @author <a href="mailto:stliu@hibernate.org">Strong Liu</a>
  */
 @RequiresDialect(H2Dialect.class)
 public class LockTimeoutPropertyTest extends BaseEntityManagerFunctionalTestCase {

hibernate-core/src/test/java/org/hibernate/jpa/test/procedure/AbstractStoredProcedureTest.java

@@ -23,7 +23,7 @@
 import static org.junit.Assert.assertNotNull;
 
 /**
- * @author Strong Liu <stliu@hibernate.org>
+ * @author <a href="mailto:stliu@hibernate.org">Strong Liu</a>
  */
 public abstract class AbstractStoredProcedureTest extends BaseEntityManagerFunctionalTestCase {
 	@Test

hibernate-core/src/test/java/org/hibernate/jpa/test/procedure/AnnotationTest.java

@@ -7,7 +7,7 @@
 package org.hibernate.jpa.test.procedure;
 
 /**
- * @author Strong Liu <stliu@hibernate.org>
+ * @author <a href="mailto:stliu@hibernate.org">Strong Liu</a>
  */
 public class AnnotationTest extends AbstractStoredProcedureTest {
 	@Override

hibernate-core/src/test/java/org/hibernate/jpa/test/procedure/OrmTest.java

@@ -7,7 +7,7 @@
 package org.hibernate.jpa.test.procedure;
 
 /**
- * @author Strong Liu <stliu@hibernate.org>
+ * @author <a href="mailto:stliu@hibernate.org">Strong Liu</a>
  */
 public class OrmTest extends AbstractStoredProcedureTest{
 	@Override