This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [RFA, 5 of 3] save/restore process record, part 5 (gcore change)


>>>>> "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.

Tom


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]