diff options
author | Tim-Philipp Müller <tim@centricular.com> | 2020-08-14 16:38:26 +0100 |
---|---|---|
committer | GStreamer Merge Bot <gitlab-merge-bot@gstreamer-foundation.org> | 2020-10-11 22:39:51 +0000 |
commit | a2cbf75523cdf8a4df1baa7007d86ef455972245 (patch) | |
tree | 450915094f51e01624e3c06c5583b04ee18b635e | |
parent | 271af223b7690e3f6f53f44684a9f57e93f5e006 (diff) |
Remove unused valgrind detection
Having this just to log a debug message in case we're
running inside valgrind doesn't seem very useful, and
the code that used to use this no longer exists it seems.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/595>
-rw-r--r-- | gst/gst_private.h | 2 | ||||
-rw-r--r-- | gst/gstinfo.c | 39 | ||||
-rw-r--r-- | meson.build | 1 |
3 files changed, 0 insertions, 42 deletions
diff --git a/gst/gst_private.h b/gst/gst_private.h index b998833247..ff75d2dcfe 100644 --- a/gst/gst_private.h +++ b/gst/gst_private.h @@ -116,8 +116,6 @@ G_GNUC_INTERNAL gboolean _priv_plugin_deps_env_vars_changed (GstPlugin * plugin G_GNUC_INTERNAL gboolean _priv_plugin_deps_files_changed (GstPlugin * plugin); -G_GNUC_INTERNAL gboolean _priv_gst_in_valgrind (void); - /* init functions called from gst_init(). */ G_GNUC_INTERNAL void _priv_gst_quarks_initialize (void); G_GNUC_INTERNAL void _priv_gst_mini_object_initialize (void); diff --git a/gst/gstinfo.c b/gst/gstinfo.c index c9a2337f55..ac6f90786e 100644 --- a/gst/gstinfo.c +++ b/gst/gstinfo.c @@ -305,36 +305,6 @@ static gboolean pretty_tags = PRETTY_TAGS_DEFAULT; static volatile gint G_GNUC_MAY_ALIAS __default_level = GST_LEVEL_DEFAULT; static volatile gint G_GNUC_MAY_ALIAS __use_color = GST_DEBUG_COLOR_MODE_ON; -/* FIXME: export this? */ -gboolean -_priv_gst_in_valgrind (void) -{ - static enum - { - GST_VG_UNCHECKED, - GST_VG_NO_VALGRIND, - GST_VG_INSIDE - } - in_valgrind = GST_VG_UNCHECKED; - - if (in_valgrind == GST_VG_UNCHECKED) { -#ifdef HAVE_VALGRIND_VALGRIND_H - if (RUNNING_ON_VALGRIND) { - GST_CAT_INFO (GST_CAT_GST_INIT, "we're running inside valgrind"); - in_valgrind = GST_VG_INSIDE; - } else { - GST_CAT_LOG (GST_CAT_GST_INIT, "not doing extra valgrind stuff"); - in_valgrind = GST_VG_NO_VALGRIND; - } -#else - in_valgrind = GST_VG_NO_VALGRIND; -#endif - g_assert (in_valgrind == GST_VG_NO_VALGRIND || - in_valgrind == GST_VG_INSIDE); - } - return (in_valgrind == GST_VG_INSIDE); -} - static gchar * _replace_pattern_in_gst_debug_file_name (gchar * name, const char *token, guint val) @@ -463,9 +433,6 @@ _priv_gst_debug_init (void) _priv_GST_CAT_PROTECTION = _gst_debug_category_new ("GST_PROTECTION", 0, "protection"); - /* print out the valgrind message if we're in valgrind */ - _priv_gst_in_valgrind (); - env = g_getenv ("GST_DEBUG_OPTIONS"); if (env != NULL) { if (strstr (env, "full_tags") || strstr (env, "full-tags")) @@ -2503,12 +2470,6 @@ gst_debug_construct_win_color (guint colorinfo) return 0; } -gboolean -_priv_gst_in_valgrind (void) -{ - return FALSE; -} - void _gst_debug_dump_mem (GstDebugCategory * cat, const gchar * file, const gchar * func, gint line, GObject * obj, const gchar * msg, diff --git a/meson.build b/meson.build index 9967d0f4c1..9167199ed7 100644 --- a/meson.build +++ b/meson.build @@ -200,7 +200,6 @@ check_headers = [ 'sys/wait.h', 'ucontext.h', 'unistd.h', - 'valgrind/valgrind.h', 'sys/resource.h', 'sys/uio.h', ] |