fix tone_alarm: avoid using exit()

On POSIX it exits the process.
This commit is contained in:
Beat Küng
2018-08-31 10:24:39 +02:00
parent 9246110161
commit fe502ffd33
4 changed files with 28 additions and 28 deletions
@@ -460,7 +460,7 @@ int tone_alarm_main(int argc, char *argv[])
if (!strcmp(argv1, "start")) { if (!strcmp(argv1, "start")) {
if (g_dev != nullptr) { if (g_dev != nullptr) {
PX4_ERR("already started"); PX4_ERR("already started");
exit(1); return 1;
} }
if (g_dev == nullptr) { if (g_dev == nullptr) {
@@ -468,33 +468,33 @@ int tone_alarm_main(int argc, char *argv[])
if (g_dev == nullptr) { if (g_dev == nullptr) {
PX4_ERR("couldn't allocate the ToneAlarm driver"); PX4_ERR("couldn't allocate the ToneAlarm driver");
exit(1); return 1;
} }
if (OK != g_dev->init()) { if (OK != g_dev->init()) {
delete g_dev; delete g_dev;
g_dev = nullptr; g_dev = nullptr;
PX4_ERR("ToneAlarm init failed"); PX4_ERR("ToneAlarm init failed");
exit(1); return 1;
} }
} }
exit(0); return 0;
} }
if (!strcmp(argv1, "stop")) { if (!strcmp(argv1, "stop")) {
delete g_dev; delete g_dev;
g_dev = nullptr; g_dev = nullptr;
exit(0); return 0;
} }
if (!strcmp(argv1, "status")) { if (!strcmp(argv1, "status")) {
g_dev->status(); g_dev->status();
exit(0); return 0;
} }
} }
tone_alarm_usage(); tone_alarm_usage();
exit(0); return 0;
} }
+7 -7
View File
@@ -456,7 +456,7 @@ int tone_alarm_main(int argc, char *argv[])
if (!strcmp(argv1, "start")) { if (!strcmp(argv1, "start")) {
if (g_dev != nullptr) { if (g_dev != nullptr) {
PX4_ERR("already started"); PX4_ERR("already started");
exit(1); return 1;
} }
if (g_dev == nullptr) { if (g_dev == nullptr) {
@@ -464,34 +464,34 @@ int tone_alarm_main(int argc, char *argv[])
if (g_dev == nullptr) { if (g_dev == nullptr) {
PX4_ERR("couldn't allocate the ToneAlarm driver"); PX4_ERR("couldn't allocate the ToneAlarm driver");
exit(1); return 1;
} }
if (OK != g_dev->init()) { if (OK != g_dev->init()) {
delete g_dev; delete g_dev;
g_dev = nullptr; g_dev = nullptr;
PX4_ERR("ToneAlarm init failed"); PX4_ERR("ToneAlarm init failed");
exit(1); return 1;
} }
} }
exit(0); return 0;
} }
if (!strcmp(argv1, "stop")) { if (!strcmp(argv1, "stop")) {
delete g_dev; delete g_dev;
g_dev = nullptr; g_dev = nullptr;
exit(0); return 0;
} }
if (!strcmp(argv1, "status")) { if (!strcmp(argv1, "status")) {
g_dev->status(); g_dev->status();
exit(0); return 0;
} }
} }
tone_alarm_usage(); tone_alarm_usage();
exit(0); return 0;
} }
#endif /* TONE_ALARM_CHANNEL */ #endif /* TONE_ALARM_CHANNEL */
+7 -7
View File
@@ -546,7 +546,7 @@ int tone_alarm_main(int argc, char *argv[])
if (!strcmp(argv1, "start")) { if (!strcmp(argv1, "start")) {
if (g_dev != nullptr) { if (g_dev != nullptr) {
PX4_ERR("already started"); PX4_ERR("already started");
exit(1); return 1;
} }
if (g_dev == nullptr) { if (g_dev == nullptr) {
@@ -554,33 +554,33 @@ int tone_alarm_main(int argc, char *argv[])
if (g_dev == nullptr) { if (g_dev == nullptr) {
PX4_ERR("couldn't allocate the ToneAlarm driver"); PX4_ERR("couldn't allocate the ToneAlarm driver");
exit(1); return 1;
} }
if (OK != g_dev->init()) { if (OK != g_dev->init()) {
delete g_dev; delete g_dev;
g_dev = nullptr; g_dev = nullptr;
PX4_ERR("ToneAlarm init failed"); PX4_ERR("ToneAlarm init failed");
exit(1); return 1;
} }
} }
exit(0); return 0;
} }
if (!strcmp(argv1, "stop")) { if (!strcmp(argv1, "stop")) {
delete g_dev; delete g_dev;
g_dev = nullptr; g_dev = nullptr;
exit(0); return 0;
} }
if (!strcmp(argv1, "status")) { if (!strcmp(argv1, "status")) {
g_dev->status(); g_dev->status();
exit(0); return 0;
} }
} }
tone_alarm_usage(); tone_alarm_usage();
exit(0); return 0;
} }
@@ -326,7 +326,7 @@ int tone_alarm_main(int argc, char *argv[])
if (!strcmp(argv1, "start")) { if (!strcmp(argv1, "start")) {
if (g_dev != nullptr) { if (g_dev != nullptr) {
PX4_ERR("already started"); PX4_ERR("already started");
exit(1); return 1;
} }
if (g_dev == nullptr) { if (g_dev == nullptr) {
@@ -334,33 +334,33 @@ int tone_alarm_main(int argc, char *argv[])
if (g_dev == nullptr) { if (g_dev == nullptr) {
PX4_ERR("couldn't allocate the ToneAlarm driver"); PX4_ERR("couldn't allocate the ToneAlarm driver");
exit(1); return 1;
} }
if (OK != g_dev->init()) { if (OK != g_dev->init()) {
delete g_dev; delete g_dev;
g_dev = nullptr; g_dev = nullptr;
PX4_ERR("ToneAlarm init failed"); PX4_ERR("ToneAlarm init failed");
exit(1); return 1;
} }
} }
exit(0); return 0;
} }
if (!strcmp(argv1, "stop")) { if (!strcmp(argv1, "stop")) {
delete g_dev; delete g_dev;
g_dev = nullptr; g_dev = nullptr;
exit(0); return 0;
} }
if (!strcmp(argv1, "status")) { if (!strcmp(argv1, "status")) {
g_dev->status(); g_dev->status();
exit(0); return 0;
} }
} }
tone_alarm_usage(); tone_alarm_usage();
exit(0); return 0;
} }