Code review updates

Format code

Author: adkian-sifive

core/src/main/scala-2/chisel3/experimental/dataview/ChiselSubtypeOf.scala

@@ -27,7 +27,6 @@ import scala.reflect.macros.blackbox.Context
   */
 sealed trait ChiselSubtypeOf[A, B]
 
-
 // return an empty tree here instead of a quasiquote for scala3 compatibility
 object ChiselSubtypeOf {
   // TODO return an empty tree here instead of a quasiquote for scala3 compatibility

core/src/main/scala-2/chisel3/experimental/hierarchy/core/Hierarchy.scala

@@ -7,7 +7,8 @@ import _root_.firrtl.annotations.IsModule
 import scala.reflect.runtime.universe.TypeTag
 
 trait Hierarchy[+A] extends HierarchyImpl[A] {
-/** Determine whether underlying proto is of type provided.
+
+  /** Determine whether underlying proto is of type provided.
     *
     * @note IMPORTANT: this function requires summoning a TypeTag[B], which will fail if B is an inner class.
     * @note IMPORTANT: this function IGNORES type parameters, akin to normal type erasure.

core/src/main/scala-3/chisel3/Aggregate.scala

@@ -52,8 +52,7 @@ sealed class Vec[T <: Data] private[chisel3] (gen: => T, length: Int)
 
   override def toString: String = super[VecImpl].toString
 
-  def apply(p: UInt)(using sourceInfo: SourceInfo): T = do_apply(p)
-  def do_apply(p: UInt)(using sourceInfo: SourceInfo): T = _applyImpl(p)
+  def apply(p: UInt)(using sourceInfo: SourceInfo): T = _applyImpl(p)
 
   /** A reduce operation in a tree like structure instead of sequentially
     * @example An adder tree

core/src/main/scala-3/chisel3/Disable.scala

@@ -3,8 +3,7 @@
 package chisel3
 
 import chisel3.internal._
-import chisel3.experimental.{OpaqueType, SourceInfo}
-import scala.collection.immutable.ListMap
+import chisel3.experimental.SourceInfo
 
 /** API for handling disabling of simulation constructs
   *

core/src/main/scala-3/chisel3/Module.scala

@@ -15,7 +15,7 @@ object Module extends ObjectModuleImpl with SourceInfoDoc {
     */
   // TODO(adkian-sifive) the callsite here explicitly passes
   // sourceInfo so it cannot be a contextual parameter
-  def apply[T <: BaseModule](bc: => T): T = _applyImpl(bc)
+  def apply[T <: BaseModule](bc:    => T): T = _applyImpl(bc)
   def do_apply[T <: BaseModule](bc: => T)(implicit sourceInfo: SourceInfo): T = apply(bc)
 }
 

core/src/main/scala-3/chisel3/VerificationStatement.scala

@@ -55,4 +55,3 @@ object cover extends VerifPrintMacrosDoc {
 
   type SourceLineInfo = (String, Int)
 }
-

core/src/main/scala-3/chisel3/experimental/hierarchy/HierarchyMarker.scala

@@ -1,3 +1,3 @@
 package chisel3.experimental
 
-trait Markers
+trait Markers

core/src/main/scala-3/chisel3/experimental/hierarchy/core/Hierarchy.scala

@@ -6,7 +6,8 @@ import chisel3.experimental.BaseModule
 import _root_.firrtl.annotations.IsModule
 
 trait Hierarchy[+A] extends HierarchyImpl[A] {
-/** Determine whether underlying proto is of type provided.
+
+  /** Determine whether underlying proto is of type provided.
     *
     * @note IMPORTANT: this function requires summoning a TypeTag[B], which will fail if B is an inner class.
     * @note IMPORTANT: this function IGNORES type parameters, akin to normal type erasure.

core/src/main/scala/chisel3/ChiselEnumImpl.scala

@@ -228,7 +228,7 @@ private[chisel3] abstract class EnumTypeImpl(private[chisel3] val factory: Chise
           //      value U is not a member of Char.
           //      An extension method was tried,
           //      but could not be fully constructed:
-          // 
+          //
           //          chisel3.fromLongToLiteral(c.toChar)
           // r := c.toChar.U
         }