Set loiter parameter mandatory

39ddedeb · Léo-Paul Géneau · 12bcca0c · 39ddedeb
Commit 39ddedeb authored Nov 15, 2022 by Léo-Paul Géneau 👾
Show whitespace changes
Inline Side-by-side

Showing with 4 additions and 6 deletions

qjs_wrapper.c qjs_wrapper.c +4 -6

No files found.
--- a/qjs_wrapper.c
+++ b/qjs_wrapper.c
@@ -457,11 +457,9 @@ static JSValue js_mavsdk_triggerParachute(JSContext *ctx, JSValueConst thisVal,
 static JSValue js_mavsdk_loiter(JSContext *ctx, JSValueConst thisVal,
                                int argc, JSValueConst *argv)
 {
-    double radius = 100;
+    double radius;
-    if (argc > 0) {
    if (JS_ToFloat64(ctx, &radius, argv[0]))
        return JS_EXCEPTION;
-    } 
    return JS_NewInt32(ctx, mavsdk_loiter((float)radius));
 }
@@ -605,7 +603,7 @@ static const JSCFunctionListEntry js_mavsdk_funcs[] = {
    JS_CFUNC_DEF("takeOff", 0, js_mavsdk_takeOff ),
    JS_CFUNC_DEF("takeOffAndWait", 0, js_mavsdk_takeOffAndWait ),
    JS_CFUNC_DEF("triggerParachute", 0, js_mavsdk_triggerParachute ),
-    JS_CFUNC_DEF("loiter", 0, js_mavsdk_loiter ),
+    JS_CFUNC_DEF("loiter", 1, js_mavsdk_loiter ),
    JS_CFUNC_DEF("setAirspeed", 1, js_mavsdk_setAirspeed ),
    JS_CFUNC_DEF("setAltitude", 1, js_mavsdk_setAltitude ),
    JS_CFUNC_DEF("setTargetCoordinates", 3, js_mavsdk_setTargetCoordinates ),