diff options
author | Robin Gareus <robin@gareus.org> | 2019-08-15 01:07:41 +0200 |
---|---|---|
committer | Robin Gareus <robin@gareus.org> | 2019-08-15 01:09:26 +0200 |
commit | 316880b9cf9eca5b2607458d123a0b9dbf0fee69 (patch) | |
tree | d146fdaf4b5b342810631e15102f47b1a415d472 /session_utils/new_empty_session.cc | |
parent | 4aedf9c3ea03b40a2166bbff4d9561d05c5b6d31 (diff) |
Don't print usage to stdout for invalid parameters
Diffstat (limited to 'session_utils/new_empty_session.cc')
-rw-r--r-- | session_utils/new_empty_session.cc | 14 |
1 files changed, 8 insertions, 6 deletions
diff --git a/session_utils/new_empty_session.cc b/session_utils/new_empty_session.cc index 2871852e1e..86fba1a324 100644 --- a/session_utils/new_empty_session.cc +++ b/session_utils/new_empty_session.cc @@ -29,7 +29,7 @@ using namespace ARDOUR; using namespace SessionUtils; -static void usage (int status) +static void usage () { // help2man compatible format (standard GNU help-text) printf (UTILNAME " - create a new empty session from the commandline.\n\n"); @@ -55,7 +55,7 @@ Examples:\n\ printf ("Report bugs to <http://tracker.ardour.org/>\n" "Website: <http://ardour.org/>\n"); - ::exit (status); + ::exit (EXIT_SUCCESS); } int main (int argc, char* argv[]) @@ -92,11 +92,12 @@ int main (int argc, char* argv[]) break; case 'h': - usage (EXIT_SUCCESS); + usage (); break; default: - usage (EXIT_FAILURE); + cerr << "Error: unrecognized option. See --help for usage information.\n"; + ::exit (EXIT_FAILURE); break; } } @@ -108,11 +109,12 @@ int main (int argc, char* argv[]) } else if (optind + 1 == argc) { snapshot_name = Glib::path_get_basename (argv[optind]); } else { - usage (EXIT_FAILURE); + cerr << "Error: Missing parameter. See --help for usage information.\n"; + ::exit (EXIT_FAILURE); } if (snapshot_name.empty ()) { - fprintf(stderr, "Error: Invalid empty session/snapshot name.\n"); + cerr << "Error: Invalid empty session/snapshot name.\n"; ::exit (EXIT_FAILURE); } |