minor code cleanup

carueda · Aug 27, 2024 · d8146c4 · d8146c4
1 parent e890ae8
commit d8146c4
Show file tree

Hide file tree

Showing 7 changed files with 6 additions and 10 deletions.
diff --git a/src/main/scala/tscfg/Struct.scala b/src/main/scala/tscfg/Struct.scala
@@ -40,7 +40,7 @@ case class Struct(
 
   def isLeaf: Boolean = members.isEmpty
 
-  def dependencies: Set[String] = {
+  private def dependencies: Set[String] = {
     tsStringValue.toSet ++ members.values.flatMap(_.dependencies)
   }
 

diff --git a/src/main/scala/tscfg/generators/java/JavaGen.scala b/src/main/scala/tscfg/generators/java/JavaGen.scala
@@ -516,7 +516,7 @@ class JavaGen(
           case DURATION(_) => s"""c.$getter"""
           case _ =>
             val (methodName, methodCall) =
-              tsConfigUtil.basicRequiredGetter(bt, path, genOpts.useDurations)
+              tsConfigUtil.basicRequiredGetter(bt, path)
             listAccessors += methodName -> javaDef(methodName)
             methodCall
         }
@@ -576,7 +576,7 @@ class JavaGen(
     case DOUBLE      => methodNames.dblA
     case BOOLEAN     => methodNames.blnA
     case SIZE        => methodNames.sizA
-    case DURATION(q) => methodNames.durA
+    case DURATION(_) => methodNames.durA
 
     case _: ObjectAbsType => name.replace('.', '_')
 

diff --git a/src/main/scala/tscfg/generators/scala/ScalaGen.scala b/src/main/scala/tscfg/generators/scala/ScalaGen.scala
@@ -707,7 +707,7 @@ private[scala] case class Getter(
           case DURATION(_) => s"""c.$getter"""
           case _ =>
             val (methodName, methodCall) =
-              tsConfigUtil.basicRequiredGetter(bt, path, genOpts.useDurations)
+              tsConfigUtil.basicRequiredGetter(bt, path)
             listAccessors += methodName -> scalaDef(methodName)
             methodCall
         }

diff --git a/src/main/scala/tscfg/generators/scala/ScalaUtil.scala b/src/main/scala/tscfg/generators/scala/ScalaUtil.scala
@@ -117,7 +117,7 @@ object ScalaUtil {
     "yield"
   )
 
-  val noArgMethodInScope: List[String] = List(
+  private val noArgMethodInScope: List[String] = List(
     "clone",
     "finalize",
     "getClass",

diff --git a/src/main/scala/tscfg/generators/tsConfigUtil.scala b/src/main/scala/tscfg/generators/tsConfigUtil.scala
@@ -33,7 +33,6 @@ object tsConfigUtil {
   def basicRequiredGetter(
       bt: BasicType,
       path: String,
-      useDurations: Boolean
   ): (String, String) = {
     val methodName = bt match {
       case STRING  => "$_reqStr"

diff --git a/src/main/scala/tscfg/model.scala b/src/main/scala/tscfg/model.scala
@@ -146,7 +146,7 @@ object model {
 
   // $COVERAGE-OFF$
   object util {
-    val IND = "    "
+    private val IND = "    "
 
     def format(typ: Type, ind: String = ""): String = typ match {
       case b: BasicType => b.toString

diff --git a/src/main/scala/tscfg/util.scala b/src/main/scala/tscfg/util.scala
@@ -41,7 +41,4 @@ object util {
   }
 
   // $COVERAGE-ON$
-
-  val TypesafeConfigClassName: String =
-    classOf[com.typesafe.config.Config].getName
 }