Merge pull request #79 from sashabjorkman/master

Fix the AudioWrapper constructor to be more like the VideoWrapper
murat-dogan · Feb 17, 2022 · 6be2add · 6be2add
2 parents fa1e4ad + a8c6a6a
commit 6be2add
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/media-audio-wrapper.cpp b/src/media-audio-wrapper.cpp
@@ -51,7 +51,7 @@ AudioWrapper::AudioWrapper(const Napi::CallbackInfo &info) : Napi::ObjectWrap<Au
     rtc::Description::Direction dir = rtc::Description::Direction::SendOnly;
 
     // optional
-    if (length > 1)
+    if (length > 0)
     {
         if (!info[0].IsString())
         {
@@ -62,7 +62,7 @@ AudioWrapper::AudioWrapper(const Napi::CallbackInfo &info) : Napi::ObjectWrap<Au
     }
 
     // ootional
-    if (length > 2)
+    if (length > 1)
     {
         if (!info[1].IsString())
         {