Avoid unnecessary casts on ConfigCursor#asObjectCursor

This avoids unnecessary casts on the `asObjectCursor` method of `ConfigCursor` by creating an already specialized cursor if we're using a `ConfigObjectSource`.
pureconfig · Nov 4, 2019 · 9e27a4b · 9e27a4b
1 parent a9e77dc
commit 9e27a4b
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/core/src/main/scala/pureconfig/ConfigSource.scala b/core/src/main/scala/pureconfig/ConfigSource.scala
@@ -93,6 +93,10 @@ final class ConfigObjectSource private (getConf: () => Result[Config]) extends C
   def value(): Result[ConfigObject] =
     config().right.flatMap(_.resolveSafe()).right.map(_.root)
 
+  // Avoids unnecessary cast on `ConfigCursor#asObjectCursor`.
+  override def cursor(): Result[ConfigCursor] =
+    value().right.map(ConfigObjectCursor(_, Nil))
+
   /**
    * Reads a `Config` from this config source. The returned config is usually unresolved, unless
    * the source forces it otherwise.