Tom Tromey wrote:
"Michael" == Michael Snyder <msnyder@vmware.com> writes:
Michael> +/* create_gcore_bfd -- helper for gcore_command (exported). */
Michael> +bfd *
Michael> +create_gcore_bfd (char *filename)
That comment is not very descriptive.
Michael> +/* write_gcore_file -- helper for gcore_command (exported). */
Michael> +
Michael> +void
Michael> +write_gcore_file (bfd *obfd)
Likewise.
Michael> Index: gcore.h
Michael> ===================================================================
Michael> RCS file: gcore.h
Michael> diff -N gcore.h
The gdb internals document says that headers should have include guards.
OK. Also, I found that I had included some code in this patch
that I didn't need -- so this version is shorter. ;-)