[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/external.c b/test/external.c
index 6ccc8b0..0ea5498 100644
--- a/test/external.c
+++ b/test/external.c
@@ -842,7 +842,7 @@ main (void)
 
     if (H5Fclose(file)<0) goto error;
     puts("All external storage tests passed.");
-    if (h5_cleanup(fapl)) {
+    if (h5_cleanup(FILENAME, fapl)) {
 	remove("extern_1a.raw");
 	remove("extern_1b.raw");
 	remove("extern_2a.raw");