[svn-r2531] Description:

    The "FILENAME" declared extern in h5test.h is not always used.
    It was used in h5_cleanup to remove temporary files created
    during tests.  Not all tests codes have used this routine.
    Indeed, quite a few of test programs do "#define FILENAME ".
    Also, h5_cleanup needs to work in tandem with h5_fixname.
    h5_fixname accepts an explicite base_name argument instead
    of using the global variable FILENAME.  That is cleaner.
Solution:
    Added char *base_name[] as a new argument to h5_cleanup, in
    the same style as h5_fixname.  Removed "extern char *FILENAME..."
    from use.  Also, undo some unnecessary declaration of "char *FILENAME"
    from some tests which don't use it at all (yet).
Platforms tested:
    modi4-64(irix64), arabica(solari2.7), eirene(linux)
    (arabica could not launch tests automatically.  I had to hack
     in LD_LIBRARY_PATH to make them run.)

1 files changed, 1 insertions, 1 deletions

diff --git a/test/fillval.c b/test/fillval.c
index 28fa576..f5f9700 100644
--- a/test/fillval.c
+++ b/test/fillval.c
@@ -736,7 +736,7 @@ main(int argc, char *argv[])
     
     if (nerrors) goto error;
     puts("All fill value tests passed.");
-    if (h5_cleanup(fapl)) remove(FILE_NAME_RAW);
+    if (h5_cleanup(FILENAME, fapl)) remove(FILE_NAME_RAW);
     return 0;
 
  error: