[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, 8 deletions

diff --git a/test/h5test.h b/test/h5test.h
index 874a5f6..1dc00c6 100644
--- a/test/h5test.h
+++ b/test/h5test.h
@@ -23,13 +23,6 @@
 
 
 /*
- * This array should contain a list of file base names created by the test.
- * The base name is a word like `test' which will have a prefix and suffix
- * added to result in something like `ufs:/u/matzke/test.h5'
- */
-extern const char *FILENAME[];
-
-/*
  * This contains the filename prefix specificied as command line option for
  * the parallel test files. 
  */
@@ -60,7 +53,7 @@ extern char *paraprefix;
 extern "C" {
 #endif
 
-int h5_cleanup(hid_t fapl);
+int h5_cleanup(const char *base_name[], hid_t fapl);
 herr_t h5_errors(void *client_data);
 char *h5_fixname(const char *base_name, hid_t fapl, char *fullname,
 		 size_t size);