resolve comments

Author: fs-eire

js/node/src/inference_session_wrap.cc

@@ -77,7 +77,7 @@ Napi::Value InferenceSessionWrap::LoadModel(const Napi::CallbackInfo& info) {
       Napi::String value = info[0].As<Napi::String>();
 
       ParseSessionOptions(info[1].As<Napi::Object>(), sessionOptions);
-      this->session_.reset(new Ort::Session(*OrtSingletonData::Env(),
+      this->session_.reset(new Ort::Session(OrtSingletonData::Env(),
 #ifdef _WIN32
                                             reinterpret_cast<const wchar_t*>(value.Utf16Value().c_str()),
 #else
@@ -92,7 +92,7 @@ Napi::Value InferenceSessionWrap::LoadModel(const Napi::CallbackInfo& info) {
       int64_t bytesLength = info[2].As<Napi::Number>().Int64Value();
 
       ParseSessionOptions(info[3].As<Napi::Object>(), sessionOptions);
-      this->session_.reset(new Ort::Session(*OrtSingletonData::Env(),
+      this->session_.reset(new Ort::Session(OrtSingletonData::Env(),
                                             reinterpret_cast<char*>(buffer) + bytesOffset, bytesLength,
                                             sessionOptions));
     } else {
@@ -212,7 +212,7 @@ Napi::Value InferenceSessionWrap::Run(const Napi::CallbackInfo& info) {
       ParseRunOptions(info[2].As<Napi::Object>(), runOptions);
     }
     if (preferredOutputLocations_.size() == 0) {
-      session_->Run(runOptions == nullptr ? *OrtSingletonData::DefaultRunOptions() : runOptions,
+      session_->Run(runOptions == nullptr ? OrtSingletonData::DefaultRunOptions() : runOptions,
                     inputIndex == 0 ? nullptr : &inputNames_cstr[0], inputIndex == 0 ? nullptr : &inputValues[0],
                     inputIndex, outputIndex == 0 ? nullptr : &outputNames_cstr[0],
                     outputIndex == 0 ? nullptr : &outputValues[0], outputIndex);
@@ -241,7 +241,7 @@ Napi::Value InferenceSessionWrap::Run(const Napi::CallbackInfo& info) {
         }
       }
 
-      session_->Run(runOptions == nullptr ? *OrtSingletonData::DefaultRunOptions() : runOptions, *ioBinding_);
+      session_->Run(runOptions == nullptr ? OrtSingletonData::DefaultRunOptions() : runOptions, *ioBinding_);
 
       auto outputs = ioBinding_->GetOutputValues();
       ORT_NAPI_THROW_ERROR_IF(outputs.size() != outputIndex, env, "Output count mismatch.");

js/node/src/ort_singleton_data.cc

@@ -4,19 +4,19 @@
 #include "ort_singleton_data.h"
 
 OrtSingletonData::OrtObjects::OrtObjects(int log_level)
-    : env(new Ort::Env{OrtLoggingLevel(log_level), "onnxruntime-node"}),
-      default_run_options(new Ort::RunOptions{}) {
+    : env{OrtLoggingLevel(log_level), "onnxruntime-node"},
+      default_run_options{} {
 }
 
 OrtSingletonData::OrtObjects& OrtSingletonData::GetOrCreateOrtObjects(int log_level) {
   static OrtObjects ort_objects(log_level);
   return ort_objects;
 }
 
-const Ort::Env* OrtSingletonData::Env() {
-  return GetOrCreateOrtObjects().env.get();
+const Ort::Env& OrtSingletonData::Env() {
+  return GetOrCreateOrtObjects().env;
 }
 
-const Ort::RunOptions* OrtSingletonData::DefaultRunOptions() {
-  return GetOrCreateOrtObjects().default_run_options.get();
+const Ort::RunOptions& OrtSingletonData::DefaultRunOptions() {
+  return GetOrCreateOrtObjects().default_run_options;
 }

js/node/src/ort_singleton_data.h

@@ -21,8 +21,8 @@
  */
 struct OrtSingletonData {
   struct OrtObjects {
-    std::unique_ptr<Ort::Env> env;
-    std::unique_ptr<Ort::RunOptions> default_run_options;
+    Ort::Env env;
+    Ort::RunOptions default_run_options;
 
    private:
     // The following pattern ensures that OrtObjects can only be created by OrtSingletonData
@@ -33,8 +33,8 @@ struct OrtSingletonData {
   static OrtObjects& GetOrCreateOrtObjects(int log_level = ORT_LOGGING_LEVEL_WARNING);
 
   // Get the global Ort::Env
-  static const Ort::Env* Env();
+  static const Ort::Env& Env();
 
   // Get the default Ort::RunOptions
-  static const Ort::RunOptions* DefaultRunOptions();
+  static const Ort::RunOptions& DefaultRunOptions();
 };