Encapsulate init in the aux constructor

PDAL · Mar 18, 2024 · 06fa549 · 06fa549
1 parent fd27b0c
commit 06fa549
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/README.md b/README.md
@@ -152,7 +152,6 @@ String json =
 
 var pipeline = new Pipeline(json, LogLevel.Error());
 
-pipeline.initialize(); // initialize the pipeline
 pipeline.execute(); // execute the pipeline
 
 var metadata = pipeline.getMetadata(); // retrieve metadata

diff --git a/core/src/main/scala/io/pdal/Pipeline.scala b/core/src/main/scala/io/pdal/Pipeline.scala
@@ -29,7 +29,9 @@ import com.github.sbt.jni.syntax.NativeLoader
 class Pipeline private (val json: String, val logLevel: Int) extends Native {
   Pipeline // reference companion object so nativeLoader loads the JNI native libraries
 
-  def this(json: String, logLevel: LogLevel.Value = LogLevel.Error) = this(json, logLevel.id)
+  def this(json: String, logLevel: LogLevel.Value = LogLevel.Error) = {
+    this(json, logLevel.id); initialize()
+  }
 
   @native def initialize(): Unit
   @native def execute(): Unit
@@ -46,7 +48,6 @@ class Pipeline private (val json: String, val logLevel: Int) extends Native {
 }
 
 object Pipeline extends NativeLoader("pdaljni.2.6") {
-  def apply(json: String, logLevel: LogLevel.Value = LogLevel.Error): Pipeline = {
-    val p = new Pipeline(json, logLevel); p.initialize(); p
-  }
+  def apply(json: String, logLevel: LogLevel.Value = LogLevel.Error): Pipeline =
+    new Pipeline(json, logLevel)
 }