From d0f57dccfbab782beb9f98cb42d9152c0dc16a0c Mon Sep 17 00:00:00 2001 From: bazooka joe Date: Fri, 30 Aug 2019 16:28:43 +0300 Subject: [PATCH 1/6] enable silent compere of parameter --- ROMFS/px4fmu_common/init.d-posix/rcS | 4 +-- ROMFS/px4fmu_common/init.d/rc.interface | 2 +- ROMFS/px4fmu_common/init.d/rc.logging | 2 +- ROMFS/px4fmu_common/init.d/rc.sensors | 18 +++++------ ROMFS/px4fmu_common/init.d/rcS | 4 +-- src/systemcmds/param/param.cpp | 40 +++++++++++++++++++++---- 6 files changed, 50 insertions(+), 20 deletions(-) diff --git a/ROMFS/px4fmu_common/init.d-posix/rcS b/ROMFS/px4fmu_common/init.d-posix/rcS index a1c629c2764f..466f4fb6d2ab 100644 --- a/ROMFS/px4fmu_common/init.d-posix/rcS +++ b/ROMFS/px4fmu_common/init.d-posix/rcS @@ -213,12 +213,12 @@ commander start navigator start -if ! param compare MNT_MODE_IN -1 +if ! param compare-silent MNT_MODE_IN -1 then vmount start fi -if param greater TRIG_MODE 0 +if param greater-silent TRIG_MODE 0 then camera_trigger start camera_feedback start diff --git a/ROMFS/px4fmu_common/init.d/rc.interface b/ROMFS/px4fmu_common/init.d/rc.interface index eaadea3e1266..e18f4b5e9567 100644 --- a/ROMFS/px4fmu_common/init.d/rc.interface +++ b/ROMFS/px4fmu_common/init.d/rc.interface @@ -17,7 +17,7 @@ set OUTPUT_DEV none # If mount (gimbal) control is enabled and output mode is AUX, set the aux # mixer to mount (override the airframe-specific MIXER_AUX setting). # -if ! param compare MNT_MODE_IN -1 +if ! param compare-silent MNT_MODE_IN -1 then if param compare MNT_MODE_OUT 0 then diff --git a/ROMFS/px4fmu_common/init.d/rc.logging b/ROMFS/px4fmu_common/init.d/rc.logging index 711cbb8087bd..3ac31d9f23c1 100644 --- a/ROMFS/px4fmu_common/init.d/rc.logging +++ b/ROMFS/px4fmu_common/init.d/rc.logging @@ -4,7 +4,7 @@ # NOTE: Script variables are declared/initialized/unset in the rcS script. # -if param greater UAVCAN_ENABLE 1 +if param greater-silent UAVCAN_ENABLE 1 then # Reduce logger buffer to free up some RAM for UAVCAN servers. set LOGGER_BUF 6 diff --git a/ROMFS/px4fmu_common/init.d/rc.sensors b/ROMFS/px4fmu_common/init.d/rc.sensors index 208175a3d45a..96fc2ba0418b 100644 --- a/ROMFS/px4fmu_common/init.d/rc.sensors +++ b/ROMFS/px4fmu_common/init.d/rc.sensors @@ -52,13 +52,13 @@ then fi fi -if param compare SENS_EN_BATT 1 +if param compare-silent SENS_EN_BATT 1 then batt_smbus start -X fi # Sensors on the PWM interface bank -if param compare SENS_EN_LL40LS 1 +if param compare-silent SENS_EN_LL40LS 1 then if pwm_input start then @@ -67,43 +67,43 @@ then fi # Lidar-Lite on I2C -if param compare SENS_EN_LL40LS 2 +if param compare-silent SENS_EN_LL40LS 2 then ll40ls start i2c fi # mappydot lidar sensor -if param compare SENS_EN_MPDT 1 +if param compare-silent SENS_EN_MPDT 1 then mappydot start -a fi # mb12xx sonar sensor -if param greater SENS_EN_MB12XX 0 +if param greater-silent SENS_EN_MB12XX 0 then mb12xx start -a fi # pga460 sonar sensor -if param greater SENS_EN_PGA460 0 +if param greater-silent SENS_EN_PGA460 0 then pga460 start fi # Lightware i2c lidar sensor -if param greater SENS_EN_SF1XX 0 +if param greater-silent SENS_EN_SF1XX 0 then sf1xx start -a fi # Heater driver for temperature regulated IMUs. -if param compare SENS_EN_THERMAL 1 +if param compare-silent SENS_EN_THERMAL 1 then heater start fi # Teraranger one tof sensor -if param greater SENS_EN_TRANGER 0 +if param greater-silent SENS_EN_TRANGER 0 then teraranger start -a fi diff --git a/ROMFS/px4fmu_common/init.d/rcS b/ROMFS/px4fmu_common/init.d/rcS index 6b83f7751326..ce6440f1e925 100644 --- a/ROMFS/px4fmu_common/init.d/rcS +++ b/ROMFS/px4fmu_common/init.d/rcS @@ -344,7 +344,7 @@ else fi # Sensors on the PWM interface bank. - if param compare SENS_EN_LL40LS 1 + if param compare-silent SENS_EN_LL40LS 1 then # Clear pins 5 and 6. set FMU_MODE pwm4 @@ -371,7 +371,7 @@ else # # Check if UAVCAN is enabled, default to it for ESCs. # - if param greater UAVCAN_ENABLE 0 + if param greater-silent UAVCAN_ENABLE 0 then # Start core UAVCAN module. if uavcan start diff --git a/src/systemcmds/param/param.cpp b/src/systemcmds/param/param.cpp index 90def735cb73..4557d79d488e 100644 --- a/src/systemcmds/param/param.cpp +++ b/src/systemcmds/param/param.cpp @@ -70,6 +70,12 @@ enum COMPARE_OPERATOR { COMPARE_OPERATOR_GREATER = 1, }; +enum COMPARE_ERROR_LEVEL { + COMPARE_ERROR_LEVEL_ERROR = 0, + COMPARE_ERROR_LEVEL_SILENT = 1, +}; + + #ifdef __PX4_QURT #define PARAM_PRINT PX4_INFO #else @@ -86,7 +92,8 @@ static int do_show_quiet(const char *param_name); static int do_show_index(const char *index, bool used_index); static void do_show_print(void *arg, param_t param); static int do_set(const char *name, const char *val, bool fail_on_not_found); -static int do_compare(const char *name, char *vals[], unsigned comparisons, enum COMPARE_OPERATOR cmd_op); +static int do_compare(const char *name, char *vals[], unsigned comparisons, enum COMPARE_OPERATOR cmd_op, + enum COMPARE_ERROR_LEVEL err_level); static int do_reset(const char *excludes[], int num_excludes); static int do_touch(const char *params[], int num_params); static int do_reset_nostart(const char *excludes[], int num_excludes); @@ -271,7 +278,17 @@ param_main(int argc, char *argv[]) if (!strcmp(argv[1], "compare")) { if (argc >= 4) { - return do_compare(argv[2], &argv[3], argc - 3, COMPARE_OPERATOR_EQUAL); + return do_compare(argv[2], &argv[3], argc - 3, COMPARE_OPERATOR_EQUAL, COMPARE_ERROR_LEVEL_ERROR); + + } else { + PX4_ERR("not enough arguments.\nTry 'param compare PARAM_NAME 3'"); + return 1; + } + } + + if (!strcmp(argv[1], "compare-silent")) { + if (argc >= 4) { + return do_compare(argv[2], &argv[3], argc - 3, COMPARE_OPERATOR_EQUAL, COMPARE_ERROR_LEVEL_SILENT); } else { PX4_ERR("not enough arguments.\nTry 'param compare PARAM_NAME 3'"); @@ -281,7 +298,17 @@ param_main(int argc, char *argv[]) if (!strcmp(argv[1], "greater")) { if (argc >= 4) { - return do_compare(argv[2], &argv[3], argc - 3, COMPARE_OPERATOR_GREATER); + return do_compare(argv[2], &argv[3], argc - 3, COMPARE_OPERATOR_GREATER, COMPARE_ERROR_LEVEL_ERROR); + + } else { + PX4_ERR("not enough arguments.\nTry 'param greater PARAM_NAME 3'"); + return 1; + } + } + + if (!strcmp(argv[1], "greater-silent")) { + if (argc >= 4) { + return do_compare(argv[2], &argv[3], argc - 3, COMPARE_OPERATOR_GREATER, COMPARE_ERROR_LEVEL_SILENT); } else { PX4_ERR("not enough arguments.\nTry 'param greater PARAM_NAME 3'"); @@ -708,7 +735,7 @@ do_set(const char *name, const char *val, bool fail_on_not_found) } static int -do_compare(const char *name, char *vals[], unsigned comparisons, enum COMPARE_OPERATOR cmp_op) +do_compare(const char *name, char *vals[], unsigned comparisons, enum COMPARE_OPERATOR cmp_op, enum COMPARE_ERROR_LEVEL err_level) { int32_t i; float f; @@ -717,7 +744,10 @@ do_compare(const char *name, char *vals[], unsigned comparisons, enum COMPARE_OP /* set nothing if parameter cannot be found */ if (param == PARAM_INVALID) { /* param not found */ - PX4_DEBUG("Parameter %s not found", name); + if(err_level == COMPARE_ERROR_LEVEL_ERROR) + { + PX4_ERR("Parameter %s not found", name); + } return 1; } From f8a2d9562cae1831f90c3b07c033d14ccfa20c35 Mon Sep 17 00:00:00 2001 From: bazooka joe Date: Fri, 30 Aug 2019 17:07:48 +0300 Subject: [PATCH 2/6] add compare-silent to MNT_MODE_OU --- ROMFS/px4fmu_common/init.d/rc.interface | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ROMFS/px4fmu_common/init.d/rc.interface b/ROMFS/px4fmu_common/init.d/rc.interface index e18f4b5e9567..fdfe6d4fcdf4 100644 --- a/ROMFS/px4fmu_common/init.d/rc.interface +++ b/ROMFS/px4fmu_common/init.d/rc.interface @@ -19,7 +19,7 @@ set OUTPUT_DEV none # if ! param compare-silent MNT_MODE_IN -1 then - if param compare MNT_MODE_OUT 0 + if param compare-silent MNT_MODE_OUT 0 then set MIXER_AUX mount fi From 1c85968672dd0b3a963eb177568a0c90e15b47ec Mon Sep 17 00:00:00 2001 From: bazooka joe Date: Tue, 3 Sep 2019 21:03:36 +0300 Subject: [PATCH 3/6] change the compare-silent to compare -s command --- ROMFS/px4fmu_common/init.d-posix/rcS | 4 +-- ROMFS/px4fmu_common/init.d/rc.interface | 4 +-- ROMFS/px4fmu_common/init.d/rc.logging | 2 +- ROMFS/px4fmu_common/init.d/rc.sensors | 18 ++++++------- ROMFS/px4fmu_common/init.d/rcS | 4 +-- src/systemcmds/param/param.cpp | 34 ++++++++----------------- 6 files changed, 26 insertions(+), 40 deletions(-) diff --git a/ROMFS/px4fmu_common/init.d-posix/rcS b/ROMFS/px4fmu_common/init.d-posix/rcS index 466f4fb6d2ab..17f00d9e2441 100644 --- a/ROMFS/px4fmu_common/init.d-posix/rcS +++ b/ROMFS/px4fmu_common/init.d-posix/rcS @@ -213,12 +213,12 @@ commander start navigator start -if ! param compare-silent MNT_MODE_IN -1 +if ! param compare -s MNT_MODE_IN -1 then vmount start fi -if param greater-silent TRIG_MODE 0 +if param greater -s TRIG_MODE 0 then camera_trigger start camera_feedback start diff --git a/ROMFS/px4fmu_common/init.d/rc.interface b/ROMFS/px4fmu_common/init.d/rc.interface index fdfe6d4fcdf4..e33882b17089 100644 --- a/ROMFS/px4fmu_common/init.d/rc.interface +++ b/ROMFS/px4fmu_common/init.d/rc.interface @@ -17,9 +17,9 @@ set OUTPUT_DEV none # If mount (gimbal) control is enabled and output mode is AUX, set the aux # mixer to mount (override the airframe-specific MIXER_AUX setting). # -if ! param compare-silent MNT_MODE_IN -1 +if ! param compare -s MNT_MODE_IN -1 then - if param compare-silent MNT_MODE_OUT 0 + if param compare -s MNT_MODE_OUT 0 then set MIXER_AUX mount fi diff --git a/ROMFS/px4fmu_common/init.d/rc.logging b/ROMFS/px4fmu_common/init.d/rc.logging index 3ac31d9f23c1..11584dfc6270 100644 --- a/ROMFS/px4fmu_common/init.d/rc.logging +++ b/ROMFS/px4fmu_common/init.d/rc.logging @@ -4,7 +4,7 @@ # NOTE: Script variables are declared/initialized/unset in the rcS script. # -if param greater-silent UAVCAN_ENABLE 1 +if param greater -s UAVCAN_ENABLE 1 then # Reduce logger buffer to free up some RAM for UAVCAN servers. set LOGGER_BUF 6 diff --git a/ROMFS/px4fmu_common/init.d/rc.sensors b/ROMFS/px4fmu_common/init.d/rc.sensors index 96fc2ba0418b..c61f76491b15 100644 --- a/ROMFS/px4fmu_common/init.d/rc.sensors +++ b/ROMFS/px4fmu_common/init.d/rc.sensors @@ -52,13 +52,13 @@ then fi fi -if param compare-silent SENS_EN_BATT 1 +if param compare -s SENS_EN_BATT 1 then batt_smbus start -X fi # Sensors on the PWM interface bank -if param compare-silent SENS_EN_LL40LS 1 +if param compare -s SENS_EN_LL40LS 1 then if pwm_input start then @@ -67,43 +67,43 @@ then fi # Lidar-Lite on I2C -if param compare-silent SENS_EN_LL40LS 2 +if param compare -s SENS_EN_LL40LS 2 then ll40ls start i2c fi # mappydot lidar sensor -if param compare-silent SENS_EN_MPDT 1 +if param compare -s SENS_EN_MPDT 1 then mappydot start -a fi # mb12xx sonar sensor -if param greater-silent SENS_EN_MB12XX 0 +if param greater -s SENS_EN_MB12XX 0 then mb12xx start -a fi # pga460 sonar sensor -if param greater-silent SENS_EN_PGA460 0 +if param greater -s SENS_EN_PGA460 0 then pga460 start fi # Lightware i2c lidar sensor -if param greater-silent SENS_EN_SF1XX 0 +if param greater -s SENS_EN_SF1XX 0 then sf1xx start -a fi # Heater driver for temperature regulated IMUs. -if param compare-silent SENS_EN_THERMAL 1 +if param compare -s SENS_EN_THERMAL 1 then heater start fi # Teraranger one tof sensor -if param greater-silent SENS_EN_TRANGER 0 +if param greater -s SENS_EN_TRANGER 0 then teraranger start -a fi diff --git a/ROMFS/px4fmu_common/init.d/rcS b/ROMFS/px4fmu_common/init.d/rcS index ce6440f1e925..621c5a899761 100644 --- a/ROMFS/px4fmu_common/init.d/rcS +++ b/ROMFS/px4fmu_common/init.d/rcS @@ -344,7 +344,7 @@ else fi # Sensors on the PWM interface bank. - if param compare-silent SENS_EN_LL40LS 1 + if param compare -s SENS_EN_LL40LS 1 then # Clear pins 5 and 6. set FMU_MODE pwm4 @@ -371,7 +371,7 @@ else # # Check if UAVCAN is enabled, default to it for ESCs. # - if param greater-silent UAVCAN_ENABLE 0 + if param greater -s UAVCAN_ENABLE 0 then # Start core UAVCAN module. if uavcan start diff --git a/src/systemcmds/param/param.cpp b/src/systemcmds/param/param.cpp index 4557d79d488e..ffb32ab811a4 100644 --- a/src/systemcmds/param/param.cpp +++ b/src/systemcmds/param/param.cpp @@ -150,10 +150,14 @@ Change the airframe and make sure the airframe's default parameters are loaded: PRINT_MODULE_USAGE_ARG("fail", "If provided, let the command fail if param is not found", true); PRINT_MODULE_USAGE_COMMAND_DESCR("compare", "Compare a param with a value. Command will succeed if equal"); + PRINT_MODULE_USAGE_PARAM_FLAG('s', "If provided, silent errors if parameter doesn't exists", true); PRINT_MODULE_USAGE_ARG(" ", "Parameter name and value to compare", false); PRINT_MODULE_USAGE_COMMAND_DESCR("greater", "Compare a param with a value. Command will succeed if param is greater than the value"); + PRINT_MODULE_USAGE_PARAM_FLAG('s', "If provided, silent errors if parameter doesn't exists", true); + PRINT_MODULE_USAGE_ARG(" ", "Parameter name and value to compare", false); + PRINT_MODULE_USAGE_ARG(" ", "Parameter name and value to compare", false); PRINT_MODULE_USAGE_COMMAND_DESCR("touch", "Mark a parameter as used"); @@ -277,19 +281,10 @@ param_main(int argc, char *argv[]) } if (!strcmp(argv[1], "compare")) { - if (argc >= 4) { + if(argc >= 5 && !strcmp(argv[2], "-s")) { + return do_compare(argv[3], &argv[4], argc - 4, COMPARE_OPERATOR_EQUAL, COMPARE_ERROR_LEVEL_SILENT); + } else if (argc >= 4) { return do_compare(argv[2], &argv[3], argc - 3, COMPARE_OPERATOR_EQUAL, COMPARE_ERROR_LEVEL_ERROR); - - } else { - PX4_ERR("not enough arguments.\nTry 'param compare PARAM_NAME 3'"); - return 1; - } - } - - if (!strcmp(argv[1], "compare-silent")) { - if (argc >= 4) { - return do_compare(argv[2], &argv[3], argc - 3, COMPARE_OPERATOR_EQUAL, COMPARE_ERROR_LEVEL_SILENT); - } else { PX4_ERR("not enough arguments.\nTry 'param compare PARAM_NAME 3'"); return 1; @@ -297,19 +292,10 @@ param_main(int argc, char *argv[]) } if (!strcmp(argv[1], "greater")) { - if (argc >= 4) { + if(argc >= 5 && !strcmp(argv[2], "-s")) { + return do_compare(argv[3], &argv[4], argc - 4, COMPARE_OPERATOR_GREATER, COMPARE_ERROR_LEVEL_SILENT); + } else if (argc >= 4) { return do_compare(argv[2], &argv[3], argc - 3, COMPARE_OPERATOR_GREATER, COMPARE_ERROR_LEVEL_ERROR); - - } else { - PX4_ERR("not enough arguments.\nTry 'param greater PARAM_NAME 3'"); - return 1; - } - } - - if (!strcmp(argv[1], "greater-silent")) { - if (argc >= 4) { - return do_compare(argv[2], &argv[3], argc - 3, COMPARE_OPERATOR_GREATER, COMPARE_ERROR_LEVEL_SILENT); - } else { PX4_ERR("not enough arguments.\nTry 'param greater PARAM_NAME 3'"); return 1; From 456de15feb5e6c9b24c014f099e89b82983cbd7d Mon Sep 17 00:00:00 2001 From: bazooka joe Date: Sun, 6 Oct 2019 19:14:26 +0300 Subject: [PATCH 4/6] silencing comperationg of parameters that might not available on some versions: SYS_USE_IO, SENS_EN_PMW3901, CAM_CAP_FBACK --- ROMFS/px4fmu_common/init.d/rc.sensors | 2 +- ROMFS/px4fmu_common/init.d/rcS | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/ROMFS/px4fmu_common/init.d/rc.sensors b/ROMFS/px4fmu_common/init.d/rc.sensors index bd78ebecfdcb..14627b587a4d 100644 --- a/ROMFS/px4fmu_common/init.d/rc.sensors +++ b/ROMFS/px4fmu_common/init.d/rc.sensors @@ -104,7 +104,7 @@ then fi # Possible pmw3901 optical flow sensor -if param greater SENS_EN_PMW3901 0 +if param greater -s SENS_EN_PMW3901 0 then pmw3901 start fi diff --git a/ROMFS/px4fmu_common/init.d/rcS b/ROMFS/px4fmu_common/init.d/rcS index 9db730c19737..846d8abd0829 100644 --- a/ROMFS/px4fmu_common/init.d/rcS +++ b/ROMFS/px4fmu_common/init.d/rcS @@ -299,7 +299,7 @@ else # # Set USE_IO flag. # - if param compare SYS_USE_IO 1 + if param compare -s SYS_USE_IO 1 then set USE_IO yes fi @@ -426,7 +426,7 @@ else sh /etc/init.d/rc.vehicle_setup # Camera capture driver - if param greater CAM_CAP_FBACK 0 + if param greater -s CAM_CAP_FBACK 0 then if camera_capture start then From e22ab17752f5ab226aa392cba835cb2f1e4bf42c Mon Sep 17 00:00:00 2001 From: bazooka joe Date: Sun, 6 Oct 2019 19:14:26 +0300 Subject: [PATCH 5/6] silencing comperationg of parameters that might not available on some versions: SYS_USE_IO, SENS_EN_PMW3901, CAM_CAP_FBACK --- ROMFS/px4fmu_common/init.d/rc.sensors | 2 +- ROMFS/px4fmu_common/init.d/rcS | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/ROMFS/px4fmu_common/init.d/rc.sensors b/ROMFS/px4fmu_common/init.d/rc.sensors index bd78ebecfdcb..14627b587a4d 100644 --- a/ROMFS/px4fmu_common/init.d/rc.sensors +++ b/ROMFS/px4fmu_common/init.d/rc.sensors @@ -104,7 +104,7 @@ then fi # Possible pmw3901 optical flow sensor -if param greater SENS_EN_PMW3901 0 +if param greater -s SENS_EN_PMW3901 0 then pmw3901 start fi diff --git a/ROMFS/px4fmu_common/init.d/rcS b/ROMFS/px4fmu_common/init.d/rcS index 9db730c19737..846d8abd0829 100644 --- a/ROMFS/px4fmu_common/init.d/rcS +++ b/ROMFS/px4fmu_common/init.d/rcS @@ -299,7 +299,7 @@ else # # Set USE_IO flag. # - if param compare SYS_USE_IO 1 + if param compare -s SYS_USE_IO 1 then set USE_IO yes fi @@ -426,7 +426,7 @@ else sh /etc/init.d/rc.vehicle_setup # Camera capture driver - if param greater CAM_CAP_FBACK 0 + if param greater -s CAM_CAP_FBACK 0 then if camera_capture start then From cc914952c5906432e2e99d27ecd3b6ee4b2eaf24 Mon Sep 17 00:00:00 2001 From: bazooka joe Date: Sun, 6 Oct 2019 20:58:22 +0300 Subject: [PATCH 6/6] changed enum to 'enum class' --- src/systemcmds/param/param.cpp | 30 +++++++++++++++--------------- 1 file changed, 15 insertions(+), 15 deletions(-) diff --git a/src/systemcmds/param/param.cpp b/src/systemcmds/param/param.cpp index ffb32ab811a4..7f143015302c 100644 --- a/src/systemcmds/param/param.cpp +++ b/src/systemcmds/param/param.cpp @@ -65,14 +65,14 @@ __BEGIN_DECLS __EXPORT int param_main(int argc, char *argv[]); __END_DECLS -enum COMPARE_OPERATOR { - COMPARE_OPERATOR_EQUAL = 0, - COMPARE_OPERATOR_GREATER = 1, +enum class COMPARE_OPERATOR { + EQUAL = 0, + GREATER = 1, }; -enum COMPARE_ERROR_LEVEL { - COMPARE_ERROR_LEVEL_ERROR = 0, - COMPARE_ERROR_LEVEL_SILENT = 1, +enum class COMPARE_ERROR_LEVEL { + DO_ERROR = 0, + SILENT = 1, }; @@ -282,9 +282,9 @@ param_main(int argc, char *argv[]) if (!strcmp(argv[1], "compare")) { if(argc >= 5 && !strcmp(argv[2], "-s")) { - return do_compare(argv[3], &argv[4], argc - 4, COMPARE_OPERATOR_EQUAL, COMPARE_ERROR_LEVEL_SILENT); + return do_compare(argv[3], &argv[4], argc - 4, COMPARE_OPERATOR::EQUAL, COMPARE_ERROR_LEVEL::SILENT); } else if (argc >= 4) { - return do_compare(argv[2], &argv[3], argc - 3, COMPARE_OPERATOR_EQUAL, COMPARE_ERROR_LEVEL_ERROR); + return do_compare(argv[2], &argv[3], argc - 3, COMPARE_OPERATOR::EQUAL, COMPARE_ERROR_LEVEL::DO_ERROR); } else { PX4_ERR("not enough arguments.\nTry 'param compare PARAM_NAME 3'"); return 1; @@ -293,9 +293,9 @@ param_main(int argc, char *argv[]) if (!strcmp(argv[1], "greater")) { if(argc >= 5 && !strcmp(argv[2], "-s")) { - return do_compare(argv[3], &argv[4], argc - 4, COMPARE_OPERATOR_GREATER, COMPARE_ERROR_LEVEL_SILENT); + return do_compare(argv[3], &argv[4], argc - 4, COMPARE_OPERATOR::GREATER, COMPARE_ERROR_LEVEL::SILENT); } else if (argc >= 4) { - return do_compare(argv[2], &argv[3], argc - 3, COMPARE_OPERATOR_GREATER, COMPARE_ERROR_LEVEL_ERROR); + return do_compare(argv[2], &argv[3], argc - 3, COMPARE_OPERATOR::GREATER, COMPARE_ERROR_LEVEL::DO_ERROR); } else { PX4_ERR("not enough arguments.\nTry 'param greater PARAM_NAME 3'"); return 1; @@ -730,7 +730,7 @@ do_compare(const char *name, char *vals[], unsigned comparisons, enum COMPARE_OP /* set nothing if parameter cannot be found */ if (param == PARAM_INVALID) { /* param not found */ - if(err_level == COMPARE_ERROR_LEVEL_ERROR) + if(err_level == COMPARE_ERROR_LEVEL::DO_ERROR) { PX4_ERR("Parameter %s not found", name); } @@ -754,8 +754,8 @@ do_compare(const char *name, char *vals[], unsigned comparisons, enum COMPARE_OP int j = strtol(vals[k], &end, 10); - if (((cmp_op == COMPARE_OPERATOR_EQUAL) && (i == j)) || - ((cmp_op == COMPARE_OPERATOR_GREATER) && (i > j))) { + if (((cmp_op == COMPARE_OPERATOR::EQUAL) && (i == j)) || + ((cmp_op == COMPARE_OPERATOR::GREATER) && (i > j))) { PX4_DEBUG(" %ld: ", (long)i); ret = 0; } @@ -774,8 +774,8 @@ do_compare(const char *name, char *vals[], unsigned comparisons, enum COMPARE_OP float g = strtod(vals[k], &end); - if (((cmp_op == COMPARE_OPERATOR_EQUAL) && (fabsf(f - g) < 1e-7f)) || - ((cmp_op == COMPARE_OPERATOR_GREATER) && (f > g))) { + if (((cmp_op == COMPARE_OPERATOR::EQUAL) && (fabsf(f - g) < 1e-7f)) || + ((cmp_op == COMPARE_OPERATOR::GREATER) && (f > g))) { PX4_DEBUG(" %4.4f: ", (double)f); ret = 0; }