Reset unique_ptr when failed at constructor

chokobole · Dec 25, 2019 · bb67f25 · bb67f25
1 parent 5ff9327
commit bb67f25
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/examples/calculator.cc b/examples/calculator.cc
@@ -77,16 +77,18 @@ void CalculatorJs::Init(Napi::Env env, Napi::Object exports) {
 
 CalculatorJs::CalculatorJs(const Napi::CallbackInfo& info)
     : Napi::ObjectWrap<CalculatorJs>(info) {
+  Napi::Env env = info.Env();
   if (info.Length() == 0) {
     calculator_ = std::unique_ptr<Calculator>(
         TypedConstruct(info, &Constructor<Calculator>::CallNew<>));
   } else if (info.Length() == 1) {
     calculator_ = std::unique_ptr<Calculator>(
         TypedConstruct(info, &Constructor<Calculator>::CallNew<int>));
   } else {
-    Napi::Env env = info.Env();
     THROW_JS_WRONG_NUMBER_OF_ARGUMENTS(env);
   }
+
+  if (env.IsExceptionPending()) calculator_.reset();
 }
 
 Napi::Object Init(Napi::Env env, Napi::Object exports) {