Move ht_sigchld_ignore and ht_ignore_node to pub_tool_signals.h

ht_sigchld_ignore and ht_ignore_node were defined in pub_core_signals.h
which cannot include any other tool header.

...checking header files and include directives
*** File coregrind/pub_core_signals.h must not include pub_tool_hashtable.h

So move the definition and type to pub_tool_signals.h

diff --git a/coregrind/pub_core_signals.h b/coregrind/pub_core_signals.h
index 1c86ee5f63cffbe55cfe11424afe2eebe67aeb80..ae8555ba8062e04098d1bec8d1bf7a68e2d077de 100644 (file)
--- a/coregrind/pub_core_signals.h
+++ b/coregrind/pub_core_signals.h
@@ -35,7 +35,6 @@
 
 #include "pub_tool_signals.h"       // I want to get rid of this header...
 #include "pub_core_vki.h"           // vki_sigset_t et al.
-#include "pub_tool_hashtable.h"
 
 /* Highest signal the kernel will let us use */
 extern Int VG_(max_signal);
@@ -86,15 +85,6 @@ extern Bool VG_(extend_stack)(ThreadId tid, Addr addr);
    before using that signal to kill the process. */
 extern void VG_(set_default_handler)(Int sig);
 
-/* Hash table of PIDs from which SIGCHLD is ignored.  */
-extern VgHashTable *ht_sigchld_ignore;
-
-/* Hash table node where each key represents a PID.  */
-typedef struct _ht_ignore_node {
-   struct _ht_ignore_node *next;
-   UWord   key;
-} ht_ignore_node;
-
 #endif   // __PUB_CORE_SIGNALS_H
 
 /*--------------------------------------------------------------------*/

diff --git a/include/pub_tool_signals.h b/include/pub_tool_signals.h
index 0de30465c1e9ec81fd79e1e5bee6c4efc57eb2a9..f037cd5d7e26ccc2ff818b23324cfd96b73d0f9b 100644 (file)
--- a/include/pub_tool_signals.h
+++ b/include/pub_tool_signals.h
@@ -30,6 +30,7 @@
 #define __PUB_TOOL_SIGNALS_H
 
 #include "pub_tool_basics.h"   // Addr
+#include "pub_tool_hashtable.h" // VgHashTable
 
 // Register an interest in apparently internal faults; used code which
 // wanders around dangerous memory (ie, leakcheck).  The catcher is
@@ -43,6 +44,15 @@
 typedef void (*fault_catcher_t)(Int sig, Addr addr);
 extern fault_catcher_t VG_(set_fault_catcher)(fault_catcher_t catcher);
 
+/* Hash table of PIDs from which SIGCHLD is ignored.  */
+extern VgHashTable *ht_sigchld_ignore;
+
+/* Hash table node where each key represents a PID.  */
+typedef struct _ht_ignore_node {
+   struct _ht_ignore_node *next;
+   UWord   key;
+} ht_ignore_node;
+
 #endif   // __PUB_TOOL_SIGNALS_H
 
 /*--------------------------------------------------------------------*/