[svn-r6007] Purpose:

    Improvement
Description:
    Moved the code parsing for missing file name in the parse_command_line
    routine.
    Added a test checking if the tool fails as expected when no file name is
    given.
Platforms tested:
    Tested h5committest {arabica (fortran), eirene (fortran, C++)
	modi4 (parallel, fortran)}? YES
Misc. update:
    Update MANIFEST if you add or remove any file. YES, added nofilename.ddl

1 files changed, 6 insertions, 10 deletions

diff --git a/tools/h5dump/h5dump.c b/tools/h5dump/h5dump.c
index 972d998..d751d25 100644
--- a/tools/h5dump/h5dump.c
+++ b/tools/h5dump/h5dump.c
@@ -2420,11 +2420,6 @@ parse_command_line(int argc, const char *argv[])
     struct handler_t   *hand, *last_dset = NULL;
     int                 i, opt, last_was_dset = FALSE;
 
-    if (argc < 2) {
-        usage(progname);
-        exit(EXIT_FAILURE);
-    }
-
     /* this will be plenty big enough to hold the info */
     hand = calloc((size_t)argc, sizeof(struct handler_t));
 
@@ -2612,6 +2607,12 @@ end_collect:
     }
 
 parse_end:
+    /* check for file name to be processed */
+    if (argc <= opt_ind) {
+        error_msg(progname, "missing file name\n");
+        usage(progname);
+        exit(EXIT_FAILURE);
+    }
     return hand;
 }
 
@@ -2694,11 +2695,6 @@ main(int argc, const char *argv[])
     struct handler_t   *hand;
     int                 i;
 
-    if (argc < 2) {
-        usage(progname);
-        exit(EXIT_FAILURE);
-    }
-
     dump_header_format = &standardformat;
     dump_function_table = &ddl_function_table;