Fixing up the Groovydocs to remove <p/> in favor of <p>

Author: pickypg

src/main/groovy/org/elasticsearch/groovy/action/ListenableActionFutureExtensions.groovy

@@ -26,7 +26,7 @@ import static com.google.common.base.Preconditions.checkNotNull
 /**
  * {@code ListenableActionFutureExtensions} adds Groovy-friendly extensions to {@link ListenableActionFuture}s to
  * enable sometimes-simpler listener support.
- * <p />
+ * <p>
  * Note: All of the listener methods behave like "add" methods. "add" was not used to avoid colliding with Java
  * variants to avoid unexpected issues ({@link Closure} is-a {@link Runnable}, which can cause confusion without explicit casting).
  * <pre>

src/main/groovy/org/elasticsearch/groovy/action/admin/cluster/repositories/put/PutRepositoryRequestExtensions.groovy

@@ -24,7 +24,7 @@ import org.elasticsearch.client.ClusterAdminClient
 
 /**
  * {@code PutRepositoryRequestExtensions} provides Groovy-friendly {@link PutRepositoryRequest} extensions.
- * <p />
+ * <p>
  * Note: This extension intentionally does _not_ provide a {@code Closure} overload of
  * {@link PutRepositoryRequest#source(java.util.Map)} because it is interpreted the same as using
  * {@link org.codehaus.groovy.runtime.DefaultGroovyMethods#with} and a {@code Closure}.

src/main/groovy/org/elasticsearch/groovy/action/admin/cluster/snapshots/create/CreateSnapshotRequestExtensions.groovy

@@ -24,7 +24,7 @@ import org.elasticsearch.client.ClusterAdminClient
 
 /**
  * {@code CreateSnapshotRequestExtensions} provides Groovy-friendly {@link CreateSnapshotRequest} extensions.
- * <p />
+ * <p>
  * Note: This extension intentionally does _not_ provide a {@code Closure} overload of
  * {@link CreateSnapshotRequest#source(java.util.Map)} because it is interpreted the same as using
  * {@link org.codehaus.groovy.runtime.DefaultGroovyMethods#with} and a {@code Closure}.

src/main/groovy/org/elasticsearch/groovy/action/admin/cluster/snapshots/restore/RestoreSnapshotRequestExtensions.groovy

@@ -24,7 +24,7 @@ import org.elasticsearch.client.ClusterAdminClient
 
 /**
  * {@code RestoreSnapshotRequestExtensions} provides Groovy-friendly {@link RestoreSnapshotRequest} extensions.
- * <p />
+ * <p>
  * Note: This extension intentionally does _not_ provide a {@code Closure} overload of
  * {@link RestoreSnapshotRequest#source(java.util.Map)} because it is interpreted the same as using
  * {@link org.codehaus.groovy.runtime.DefaultGroovyMethods#with} and a {@code Closure}.

src/main/groovy/org/elasticsearch/groovy/action/search/SearchRequestExtensions.groovy

@@ -88,7 +88,7 @@ class SearchRequestExtensions {
 
     /**
      * Constructs a new search source builder with a raw search query.
-     * <p />
+     * <p>
      * Note: When building a new {@link SearchRequest}, using this method will overwrite other changes related to the
      * query.
      *

src/main/groovy/org/elasticsearch/groovy/client/ClientExtensions.groovy

@@ -121,7 +121,7 @@ class ClientExtensions extends AbstractClientExtensions {
 
     /**
      * Index a document associated with a given index and type, then get the future result.
-     * <p/>
+     * <p>
      * The id is optional. If it is not provided, one will be generated automatically.
      * <pre>
      * IndexResponse response = client.index {
@@ -152,7 +152,7 @@ class ClientExtensions extends AbstractClientExtensions {
 
     /**
      * Index a document associated with a given index and type, then get the future result.
-     * <p/>
+     * <p>
      * The id is optional. If it is not provided, one will be generated automatically.
      * <pre>
      * IndexResponse response = client.indexAsync {

src/main/groovy/org/elasticsearch/groovy/client/ClusterAdminClientExtensions.groovy

@@ -143,7 +143,7 @@ class ClusterAdminClientExtensions extends AbstractClientExtensions {
 
     /**
      * Reroute the allocation of shards in the cluster.
-     * <p />
+     * <p>
      * Note: This is an Advanced API and care should be taken before performing related operations.
      *
      * @param self The {@code this} reference for the {@link ClusterAdminClient}.
@@ -157,7 +157,7 @@ class ClusterAdminClientExtensions extends AbstractClientExtensions {
 
     /**
      * Reroute the allocation of shards in the cluster.
-     * <p />
+     * <p>
      * Note: This is an Advanced API and care should be taken before performing related operations.
      *
      * @param self The {@code this} reference for the {@link ClusterAdminClient}.
@@ -482,7 +482,7 @@ class ClusterAdminClientExtensions extends AbstractClientExtensions {
 
     /**
      * Delete snapshots from a repository.
-     * <p />
+     * <p>
      * Note: Deleting a snapshot should only be done after creating newer snapshots, which are themselves backed up, in
      * order to avoid data loss.
      *
@@ -515,7 +515,7 @@ class ClusterAdminClientExtensions extends AbstractClientExtensions {
 
     /**
      * Get a list of pending cluster tasks that are scheduled to be executed.
-     * <p />
+     * <p>
      * This includes operations that update the cluster state (e.g., a create index operation).
      *
      * @param self The {@code this} reference for the {@link ClusterAdminClient}.

src/main/groovy/org/elasticsearch/groovy/client/IndicesAdminClientExtensions.groovy

@@ -264,7 +264,7 @@ class IndicesAdminClientExtensions extends AbstractClientExtensions {
 
     /**
      * Delete the specified indices.
-     * <p />
+     * <p>
      * Note: Manually supply the reserved index name of "_all" to delete all indices.
      *
      * @param self The {@code this} reference for the {@link IndicesAdminClient}.
@@ -279,7 +279,7 @@ class IndicesAdminClientExtensions extends AbstractClientExtensions {
 
     /**
      * Delete the specified indices.
-     * <p />
+     * <p>
      * Note: Manually supply the reserved index name of "_all" to delete all indices.
      *
      * @param self The {@code this} reference for the {@link IndicesAdminClient}.
@@ -349,7 +349,7 @@ class IndicesAdminClientExtensions extends AbstractClientExtensions {
     /**
      * Explicitly flush the specified indices. A successful flush of an index guarantees that items in its transaction
      * log have been written to disk and starts a new transaction log.
-     * <p />
+     * <p>
      * Note: By default, Elasticsearch will perform flush operations automatically.
      *
      * @param self The {@code this} reference for the {@link IndicesAdminClient}.
@@ -364,7 +364,7 @@ class IndicesAdminClientExtensions extends AbstractClientExtensions {
     /**
      * Explicitly flush the specified indices. A successful flush of an index guarantees that items in its transaction
      * log have been written to disk and starts a new transaction log.
-     * <p />
+     * <p>
      * Note: By default, Elasticsearch will perform flush operations automatically.
      *
      * @param self The {@code this} reference for the {@link IndicesAdminClient}.
@@ -378,10 +378,10 @@ class IndicesAdminClientExtensions extends AbstractClientExtensions {
 
     /**
      * Explicitly optimize the specified indices.
-     * <p />
+     * <p>
      * Optimizing an index will reduce the number of segments that the index contains, which will speed up future search
      * operations. Like other operations, Elasticsearch will automatically optimize indices in the background.
-     * <p />
+     * <p>
      * The optimal number of segments is <tt>1</tt>, but an active index will regularly have more than <tt>1</tt>. A
      * {@link IndicesAdminClient#close(CloseIndexRequest) closed} index can be safely optimized to <tt>1</tt> segment to
      * speed up future search operations.
@@ -397,10 +397,10 @@ class IndicesAdminClientExtensions extends AbstractClientExtensions {
 
     /**
      * Explicitly optimize the specified indices.
-     * <p />
+     * <p>
      * Optimizing an index will reduce the number of segments that the index contains, which will speed up future search
      * operations. Like other operations, Elasticsearch will automatically optimize indices in the background.
-     * <p />
+     * <p>
      * The optimal number of segments is <tt>1</tt>, but an active index will regularly have more than <tt>1</tt>. A
      * {@link IndicesAdminClient#close(CloseIndexRequest) closed} index can be safely optimized to <tt>1</tt> segment to
      * speed up future search operations.
@@ -588,7 +588,7 @@ class IndicesAdminClientExtensions extends AbstractClientExtensions {
 
     /**
      * Update the settings of one or more indices.
-     * <p />
+     * <p>
      * Note: Some settings can only be set at the creation of an index, such as the number of shards.
      *
      * @param self The {@code this} reference for the {@link IndicesAdminClient}.
@@ -602,7 +602,7 @@ class IndicesAdminClientExtensions extends AbstractClientExtensions {
 
     /**
      * Update the settings of one or more indices.
-     * <p />
+     * <p>
      * Note: Some settings can only be set at the creation of an index, such as the number of shards.
      *
      * @param self The {@code this} reference for the {@link IndicesAdminClient}.
@@ -644,7 +644,7 @@ class IndicesAdminClientExtensions extends AbstractClientExtensions {
 
     /**
      * Delete an index template.
-     * <p />
+     * <p>
      * Note: This will <em>not</em> unmap indices that have made use of this template.
      *
      * @param self The {@code this} reference for the {@link IndicesAdminClient}.
@@ -658,7 +658,7 @@ class IndicesAdminClientExtensions extends AbstractClientExtensions {
 
     /**
      * Delete an index template.
-     * <p />
+     * <p>
      * Note: This will <em>not</em> unmap indices that have made use of this template.
      *
      * @param self The {@code this} reference for the {@link IndicesAdminClient}.

src/main/groovy/org/elasticsearch/groovy/common/settings/SettingsBuilderExtensions.groovy

@@ -24,7 +24,7 @@ import org.elasticsearch.common.settings.loader.JsonSettingsLoader
 
 /**
  * {@code ImmutableSettingsBuilderExtensions} provide Groovy-friendly extensions to {@link ImmutableSettings.Builder}.
- * <p />
+ * <p>
  * In particular, this adds the ability to specify settings in the form of a {@link Closure} in addition to existing
  * options.
  */

src/main/groovy/org/elasticsearch/groovy/common/xcontent/XContentBuilderExtensions.groovy

@@ -29,7 +29,7 @@ import org.elasticsearch.common.xcontent.XContentType
 /**
  * {@link XContentBuilderExtensions} adds Groovy-friendly extensions to {@link XContentBuilder} as well as directly to
  * {@link Closure}s.
- * <p />
+ * <p>
  * In particular, this adds the ability to convert a {@link Closure} into a {@link Map} to a {@link XContentBuilder} to
  * enable effortless conversion:
  * <pre>
@@ -168,7 +168,7 @@ class XContentBuilderExtensions {
 
     /**
      * Get the internal {@link XContentGenerator} of the {@link XContentBuilder}.
-     * <p />
+     * <p>
      * Note: This does <em>not</em> close the {@link XContentBuilder#generator()}.
      *
      * @param self The {@code this} reference for the {@link XContentBuilder}
@@ -182,7 +182,7 @@ class XContentBuilderExtensions {
 
     /**
      * Get the internal {@link OutputStream} of the {@link XContentBuilder}.
-     * <p />
+     * <p>
      * Note: This does <em>not</em> close the {@link XContentBuilder#generator()}.
      *
      * @param self The {@code this} reference for the {@link XContentBuilder}