This is the mail archive of the
gdb-patches@sources.redhat.com
mailing list for the GDB project.
[patch/rfc] Remove ``override_*'' variables from gcore.c
- From: Andrew Cagney <ac131313 at ges dot redhat dot com>
- To: gdb-patches at sources dot redhat dot com
- Date: Tue, 06 Aug 2002 12:17:58 -0400
- Subject: [patch/rfc] Remove ``override_*'' variables from gcore.c
Hello,
To follow up:
http://sources.redhat.com/ml/gdb-patches/2002-08/msg00082.html
This patch removes the override_* variables from gcore.c. If it turns
out that something needs to override the current mechanism then
something involving gdbarch or the target will be needed.
I'll look to check it in in a few days.
Andrew
2002-08-06 Andrew Cagney <ac131313@redhat.com>
* gcore.c (override_derive_stack_segment): Delete variable.
(preempt_derive_stack_segment): Delete function.
(derive_stack_segment): Delete function.
(default_derive_stack_segment): Renamed to derive_stack_segment.
(override_derive_heap_segment): Delete variable.
(preempt_derive_heap_segment): Delete function.
(derive_heap_segment): Delete function.
(default_derive_heap_segment): Rename to derive_heap_segment.
Index: gcore.c
===================================================================
RCS file: /cvs/src/src/gdb/gcore.c,v
retrieving revision 1.6
diff -u -r1.6 gcore.c
--- gcore.c 6 Aug 2002 15:53:08 -0000 1.6
+++ gcore.c 6 Aug 2002 16:11:36 -0000
@@ -157,25 +157,14 @@
return bfd_get_target (exec_bfd);
}
-/*
- * Default method for stack segment (preemptable by target).
- */
+/* Function: derive_stack_segment
-static int (*override_derive_stack_segment) (bfd_vma *, bfd_vma *);
-
-extern void
-preempt_derive_stack_segment (int (*override_func) (bfd_vma *, bfd_vma *))
-{
- override_derive_stack_segment = override_func;
-}
-
-/* Function: default_derive_stack_segment
Derive a reasonable stack segment by unwinding the target stack.
Returns 0 for failure, 1 for success. */
static int
-default_derive_stack_segment (bfd_vma *bottom, bfd_vma *top)
+derive_stack_segment (bfd_vma *bottom, bfd_vma *top)
{
bfd_vma tmp_vma;
struct frame_info *fi, *tmp_fi;
@@ -214,36 +203,15 @@
return 1; /* success */
}
-static int
-derive_stack_segment (bfd_vma *bottom, bfd_vma *top)
-{
- if (override_derive_stack_segment)
- return override_derive_stack_segment (bottom, top);
- else
- return default_derive_stack_segment (bottom, top);
-}
-
-/*
- * Default method for heap segment (preemptable by target).
- */
-
-static int (*override_derive_heap_segment) (bfd *, bfd_vma *, bfd_vma *);
-
-extern void
-preempt_derive_heap_segment (int (*override_func) (bfd *,
- bfd_vma *, bfd_vma *))
-{
- override_derive_heap_segment = override_func;
-}
+/* Function: derive_heap_segment
-/* Function: default_derive_heap_segment
Derive a reasonable heap segment by looking at sbrk and
the static data sections.
Returns 0 for failure, 1 for success. */
static int
-default_derive_heap_segment (bfd *abfd, bfd_vma *bottom, bfd_vma *top)
+derive_heap_segment (bfd *abfd, bfd_vma *bottom, bfd_vma *top)
{
bfd_vma top_of_data_memory = 0;
bfd_vma top_of_heap = 0;
@@ -305,15 +273,6 @@
}
else
return 0; /* No additional heap space needs to be saved. */
-}
-
-static int
-derive_heap_segment (bfd *abfd, bfd_vma *bottom, bfd_vma *top)
-{
- if (override_derive_heap_segment)
- return override_derive_heap_segment (abfd, bottom, top);
- else
- return default_derive_heap_segment (abfd, bottom, top);
}
/* ARGSUSED */