fix: Resolve compilation warnings related to Javadoc formatting and comment placement

Addresses issue #20719 by fixing multiple categories of compiler warnings:
- Standardized comment separator formatting to comply with style guidelines
- Repositioned annotations to appear after Javadoc comments per Java best practices
- Corrected typos in documentation comments
- Removed incomplete or orphaned comment fragments

Author: rbair23

hedera-node/hapi-utils/src/test/java/com/hedera/node/app/hapi/utils/keys/KeyComparatorTest.java

@@ -75,9 +75,9 @@ void verifyUnsupportedTypes(final String caseName, final Key lValue, final Key r
                 .isInstanceOf(UnsupportedOperationException.class);
     }
 
-    ////////////////////////////////////////////////////////////////////////////////////////////////
-    //                              Parameterized test method sources                             //
-    ////////////////////////////////////////////////////////////////////////////////////////////////
+    // ----------------------------------------------------------------------------------------------------
+    //                              Parameterized test method sources
+    // ----------------------------------------------------------------------------------------------------
     public static Stream<Arguments> generateAllCrossTypeMatches() {
         final Integer zero = Integer.valueOf(0);
         final Integer one = Integer.valueOf(1);
@@ -508,9 +508,9 @@ private static void delegateIdCases(final List<Arguments> result) {
         addDelegateId(result, minusOne, new IdVal("0.1.one", 0, 0, 1, -1), new IdVal("0.2.five", 4, 0, 2, -1));
     }
 
-    ////////////////////////////////////////////////////////////////////////////////////////////////
-    //                             Supporting methods and record types                            //
-    ////////////////////////////////////////////////////////////////////////////////////////////////
+    // ----------------------------------------------------------------------------------------------------
+    //                             Supporting methods and record types
+    // ----------------------------------------------------------------------------------------------------
     private static record IdVal(String name, int index, int realm, int shard, int num) {}
 
     private static record NamedKeyList(String name, Key key) {}
@@ -540,9 +540,9 @@ private static Key modifyId(
         else return Key.newBuilder().contractID(builder).build();
     }
 
-    ////////////////////////////////////////////////////////////////////////////////////////////////
-    //                                   Static data fragments                                    //
-    ////////////////////////////////////////////////////////////////////////////////////////////////
+    // ----------------------------------------------------------------------------------------------------
+    //                                   Static data fragments
+    // ----------------------------------------------------------------------------------------------------
     // Test ECDSA SECP256 K1 key hex values
     private static final Bytes[] ECDSA_BYTES = {
         Bytes.fromHex("00badcadfaddad2bedfedbeef959feedbeadcafecadecedebeed4acedecada5ada"),

hedera-node/hedera-app/src/main/java/com/hedera/node/app/records/impl/producers/formats/v6/BlockRecordWriterV6.java

@@ -74,7 +74,6 @@ private enum State {
     private final Signer signer;
     /** The maximum size of a sidecar file in bytes. */
     private final int maxSideCarSizeInBytes;
-    /** Whether to compress the record file and sidecar files. */
     /** The node-specific path to the directory where record files are written */
     private final Path nodeScopedRecordDir;
     /**

hedera-node/test-clients/src/main/java/com/hedera/services/bdd/suites/contract/opcodes/SStoreSuite.java

@@ -35,8 +35,8 @@
 import org.junit.jupiter.api.DynamicTest;
 import org.junit.jupiter.api.Tag;
 
-@Tag(SMART_CONTRACT)
 /** - CONCURRENCY STATUS - . Can run concurrent without temporarySStoreRefundTest() */
+@Tag(SMART_CONTRACT)
 public class SStoreSuite {
 
     private static final Logger log = LogManager.getLogger(SStoreSuite.class);

hedera-node/test-clients/src/main/java/com/hedera/services/bdd/suites/token/batch/AtomicTokenCreateSpecs.java

@@ -105,14 +105,14 @@
 
 // This test cases are direct copies of TokenCreateSpecs. The difference here is that
 // we are wrapping the operations in an atomic batch to confirm that everything works as expected.
-@HapiTestLifecycle
 /**
  * Validates the {@code TokenCreate} transaction, including its:
  * <ul>
  *     <li>Auto-association behavior.</li>
  *     <li>Default values.</li>
  * </ul>
  */
+@HapiTestLifecycle
 @Tag(TOKEN)
 class AtomicTokenCreateSpecs {
 

platform-sdk/platform-apps/demos/StatsDemo/src/main/java/com/swirlds/demo/stats/StatsDemoMain.java

@@ -88,7 +88,7 @@ public StatsDemoMain() {
                 .build();
     }
 
-    /////////////////////////////////////////////////////////////////////
+    // ----------------------------------------------------------------------------------------------------
     /** the time of the last call to preEvent */
     long lastEventTime = System.nanoTime();
     /** number of events needed to be created (the non-integer leftover from last preEvent call */

platform-sdk/platform-apps/tests/PlatformTestingTool/src/main/java/com/swirlds/demo/platform/PayloadDistribution.java

@@ -26,7 +26,7 @@ public class PayloadDistribution {
     private PAYLOAD_TYPE[] typeDistribution;
     private int[] sizeDistribution;
     private float[] ratioDistribution;
-    /** probability of payload distribution, sum of array should be 100 */
+    // probability of payload distribution, sum of array should be 100
 
     // private PayloadDistribution(Builder builder) {
     // this.typeDistribution = builder.typeDistribution;

platform-sdk/platform-apps/tests/PlatformTestingTool/src/main/java/com/swirlds/demo/platform/TransactionSubmitter.java

@@ -90,16 +90,16 @@ public static SUBMIT_GOAL fromOrdinal(int n) {
     float c2cLatencySecond = 3;
 
     /**
-     * track statistics to achive system goal
+     * track statistics to achieve system goal
      */
     long accumulatedBytes = 0;
 
     long accumulatedTrans = 0;
     long accumulatedEvent = 0;
     long accumulatedRound = 0;
 
-    long cycleStartMS = 0;
     /** cycle start time in millisecond unit */
+    long cycleStartMS = 0;
 
     /**
      * pause submission after certain amount of transaction

platform-sdk/swirlds-benchmarks/src/jmh/java/com/swirlds/benchmark/reconnect/StateBuilder.java

@@ -14,9 +14,9 @@
  * A utility class to help build random states.
  */
 public record StateBuilder(
-        /** Build a key for index 1..size. */
+        // Build a key for index 1..size.
         Function<Long, Bytes> keyBuilder,
-        /** Build a value for key index 1..size. */
+        // Build a value for key index 1..size.
         Function<Long, BenchmarkValue> valueBuilder) {
 
     /** Return {@code true} with the given probability. */

platform-sdk/swirlds-fcqueue/src/main/java/com/swirlds/fcqueue/FCQueue.java

@@ -244,9 +244,9 @@ public boolean isSelfHashing() {
         return true;
     }
 
-    //////////////////////////////////////////////////////////////////////////////////////////////////
+    // ----------------------------------------------------------------------------------------------------
     // the following implement Queue<E>
-    //////////////////////////////////////////////////////////////////////////////////////////////////
+    // ----------------------------------------------------------------------------------------------------
 
     /**
      * Inserts the specified element into this queue if it is possible to do so
@@ -392,9 +392,9 @@ public synchronized E peek() {
         return head.element;
     }
 
-    //////////////////////////////////////////////////////////////////////////////////////////////////
+    // ----------------------------------------------------------------------------------------------------
     // the following implement FastCopyable
-    //////////////////////////////////////////////////////////////////////////////////////////////////
+    // ----------------------------------------------------------------------------------------------------
 
     /** {@inheritDoc} */
     @Override
@@ -411,9 +411,9 @@ public synchronized FCQueue<E> copy() {
         return queue;
     }
 
-    //////////////////////////////////////////////////////////////////////////////////////////////////
+    // ----------------------------------------------------------------------------------------------------
     // the following implement AbstractMerkleNode
-    //////////////////////////////////////////////////////////////////////////////////////////////////
+    // ----------------------------------------------------------------------------------------------------
 
     @Override
     protected synchronized void destroyNode() {
@@ -423,9 +423,9 @@ protected synchronized void destroyNode() {
         hash = NULL_HASH;
     }
 
-    //////////////////////////////////////////////////////////////////////////////////////////////////
+    // ----------------------------------------------------------------------------------------------------
     // the following implement Java.util.Collection
-    //////////////////////////////////////////////////////////////////////////////////////////////////
+    // ----------------------------------------------------------------------------------------------------
 
     /**
      * Returns the number of elements in this collection.
@@ -717,9 +717,9 @@ public synchronized void clear() {
         hash = null;
     }
 
-    //////////////////////////////////////////////////////////////////////////////////////////////////
+    // ----------------------------------------------------------------------------------------------------
     // the following FastCopyable methods have default implementations, but are overridden here
-    //////////////////////////////////////////////////////////////////////////////////////////////////
+    // ----------------------------------------------------------------------------------------------------
 
     @Override
     public boolean equals(final Object o) {

platform-sdk/swirlds-fcqueue/src/test/java/com/swirlds/fcqueue/MockFCQueue.java

@@ -82,9 +82,9 @@ public Hash getHash() {
         } // sync
     }
 
-    //////////////////////////////////////////////////////////////////////////////////////////////////
+    // ----------------------------------------------------------------------------------------------------
     // the following implement Queue<E>
-    //////////////////////////////////////////////////////////////////////////////////////////////////
+    // ----------------------------------------------------------------------------------------------------
 
     /**
      * Inserts the specified element into this queue if it is possible to do so
@@ -280,9 +280,9 @@ public E remove() {
         }
     }
 
-    //////////////////////////////////////////////////////////////////////////////////////////////////
+    // ----------------------------------------------------------------------------------------------------
     // the following implement FastCopyable
-    //////////////////////////////////////////////////////////////////////////////////////////////////
+    // ----------------------------------------------------------------------------------------------------
 
     /** {@inheritDoc} */
     @Override