From 214e9c8244bfca72a319694ae222727f658ca6c1 Mon Sep 17 00:00:00 2001 From: Morten Fyhn Amundsen Date: Wed, 27 Nov 2019 16:11:23 +0100 Subject: [PATCH] ll40ls: Fix rotation argument parse bug Before (introduced in 7b16c3482d8), there was no colon after the `R` argument in the options specification string (ab:R). The R should be followed by a colon, because in indicates that the R option requires an argument, which it does. So I added a colon. --- src/drivers/distance_sensor/ll40ls/ll40ls.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/drivers/distance_sensor/ll40ls/ll40ls.cpp b/src/drivers/distance_sensor/ll40ls/ll40ls.cpp index af1153e6dc..c1ab519f8c 100644 --- a/src/drivers/distance_sensor/ll40ls/ll40ls.cpp +++ b/src/drivers/distance_sensor/ll40ls/ll40ls.cpp @@ -276,7 +276,7 @@ extern "C" __EXPORT int ll40ls_main(int argc, char *argv[]) bool start_i2c_all = false; bool start_pwm = false; - while ((ch = px4_getopt(argc, argv, "ab:R", &myoptind, &myoptarg)) != EOF) { + while ((ch = px4_getopt(argc, argv, "ab:R:", &myoptind, &myoptarg)) != EOF) { switch (ch) { case 'a': start_i2c_all = true;