This is the mail archive of the gdb@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: record mode supposed to work on x86_64-linux?


On Sunday 17 May 2009 01:15:00, Hui Zhu wrote:
> Hi Pedro,
> 
> What do you think about this one?

I think it's good, thanks.

> 2009-05-17  Hui Zhu  <teawater@gmail.com>
>         
>         * i386-tdep.c (i386_process_record): Remove the static of it.
>         (i386_gdbarch_init): Remove the set_gdbarch_process_record.
>         * i386-tdep.c (i386_process_record): New extern.
>         * i386-linux-tdep.c (i386_linux_init_abi): Call
>         set_gdbarch_process_record.

Please consider the alternative ChangeLog suggestions below instead.

> 	* i386-tdep.c (i386_process_record): Remove the static of it.

 	* i386-tdep.c (i386_process_record): Make it extern.

> 	(i386_gdbarch_init): Remove the set_gdbarch_process_record.

From this entry, it isn't clear what you're removing.  Actually, it
sounds like you're removing the function implementation.  Consider
instead:

	(i386_gdbarch_init): Remove the set_gdbarch_process_record call.
                                                                   ^^^^

or even:

	(i386_gdbarch_init): Don't call set_gdbarch_process_record here.


> 	* i386-tdep.c (i386_process_record): New extern.

                    ^ typo

 	* i386-tdep.h (i386_process_record): Declare.

(`new extern' sounds like adding a new variable or function,
but you're just changing the linkage of an existing function.)

-- 
Pedro Alves


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