Move VG_(set_fault_catcher) out of tool.h. Had to create a

pub_tool_signals.h for this which I didn't want to, but I couldn't
see how else to do it.

git-svn-id: svn://svn.valgrind.org/valgrind/trunk@3937

diff --git a/coregrind/pub_core_signals.h b/coregrind/pub_core_signals.h
index 444694d7f2002dda5c5f8c43bb608122dcdf2014..99ccca415d123f154b5f8fed788e49f2784d62de 100644 (file)
--- a/coregrind/pub_core_signals.h
+++ b/coregrind/pub_core_signals.h
@@ -35,6 +35,8 @@
 // PURPOSE: This module implements all the signal handling stuff.
 //--------------------------------------------------------------------
 
+#include "pub_tool_signals.h"       // I want to get rid of this header...
+
 /* Highest signal the kernel will let us use */
 extern Int VG_(max_signal);
 

diff --git a/include/pub_tool_signals.h b/include/pub_tool_signals.h
new file mode 100644 (file)
index 0000000..32deda3
--- /dev/null
+++ b/include/pub_tool_signals.h
@@ -0,0 +1,47 @@
+
+/*--------------------------------------------------------------------*/
+/*--- Signals stuff.                            pub_tool_signals.h ---*/
+/*--------------------------------------------------------------------*/
+
+/*
+   This file is part of Valgrind, a dynamic binary instrumentation
+   framework.
+
+   Copyright (C) 2000-2005 Julian Seward
+      jseward@acm.org
+
+   This program is free software; you can redistribute it and/or
+   modify it under the terms of the GNU General Public License as
+   published by the Free Software Foundation; either version 2 of the
+   License, or (at your option) any later version.
+
+   This program is distributed in the hope that it will be useful, but
+   WITHOUT ANY WARRANTY; without even the implied warranty of
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+   General Public License for more details.
+
+   You should have received a copy of the GNU General Public License
+   along with this program; if not, write to the Free Software
+   Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA
+   02111-1307, USA.
+
+   The GNU General Public License is contained in the file COPYING.
+*/
+
+#ifndef __PUB_TOOL_SIGNALS_H
+#define __PUB_TOOL_SIGNALS_H
+
+// Register an interest in apparently internal faults; used code which
+// wanders around dangerous memory (ie, leakcheck).  The catcher is
+// not expected to return.
+//
+// It's frustrating that we need this header for a single function used
+// only by Memcheck and Addrcheck during leak checking.  We should find
+// a way to remove the need for this file.
+extern void VG_(set_fault_catcher)(void (*catcher)(Int sig, Addr addr));
+
+#endif   // __PUB_TOOL_SIGNALS_H
+
+/*--------------------------------------------------------------------*/
+/*--- end                                                          ---*/
+/*--------------------------------------------------------------------*/

diff --git a/include/tool.h b/include/tool.h
index 8c5fbc8b3edbf686f42973c2120e4a66ec48cf00..f78f40da16ef1aaaeb4191a67b9d111dc706cfa6 100644 (file)
--- a/include/tool.h
+++ b/include/tool.h
@@ -72,12 +72,6 @@ extern ThreadId VG_(first_matching_thread_stack)
 /*=== Valgrind's version of libc                                   ===*/
 /*====================================================================*/
 
-/* ------------------------------------------------------------------ */
-/* Register an interest in apparently internal faults; used code which
-   wanders around dangerous memory (ie, leakcheck).  The catcher is
-   not expected to return. */
-extern void VG_(set_fault_catcher)(void (*catcher)(Int sig, Addr addr));
-
 /* ------------------------------------------------------------------ */
 /* other, randomly useful functions */
 extern Bool VG_(has_cpuid) ( void );

diff --git a/memcheck/mac_leakcheck.c b/memcheck/mac_leakcheck.c
index 09cad7023c71e00caea9490e9a21c29cec5466a4..f8bda97a4991fbf0ac1844d69e767f10e02b9516 100644 (file)
--- a/memcheck/mac_leakcheck.c
+++ b/memcheck/mac_leakcheck.c
@@ -38,6 +38,7 @@
 #include "pub_tool_libcprint.h"
 #include "pub_tool_libcsignal.h"
 #include "pub_tool_machine.h"
+#include "pub_tool_signals.h"
 
 /* Define to debug the memory-leak-detector. */
 #define VG_DEBUG_LEAKCHECK 0