[svn-r11733] Purpose:

    Code cleanup

Description:
    Added some comments about the thread-specific memory for the error and
function stacks.

    Changed H5close() to not use the function stack (since it causes it to
be re-enabled after shutting it down)

    Changed thread-safe semaphore code to not use dynamicly allocated memory.

Platforms tested:
    FreeBSD 4.11 (sleipnir)
    Linux 2.4
    Too minor to require h5committest

1 files changed, 8 insertions, 0 deletions

diff --git a/src/H5E.c b/src/H5E.c
index a3a363c..261a496 100644
--- a/src/H5E.c
+++ b/src/H5E.c
@@ -317,10 +317,18 @@ H5E_get_stack(void)
     if (!estack) {
         /* no associated value with current thread - create one */
         estack = (H5E_t *)H5FL_MALLOC(H5E_t);
+        HDassert(estack);
+
+        /* Set the thread-specific info */
         estack->nused = 0;
         estack->new_api = TRUE;
         estack->u.func_stack = (H5E_auto_stack_t)H5Eprint_stack;
         estack->auto_data = NULL;
+
+        /* (It's not necessary to release this in this API, it is
+         *      released by the "key destructor" set up in the H5TS
+         *      routines.  See calls to pthread_key_create() in H5TS.c -QAK)
+         */
         pthread_setspecific(H5TS_errstk_key_g, (void *)estack);
     }