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] Submit process record and replay fourth time, 7/8


On Mon, Mar 23, 2009 at 05:22, Mark Kettenis <mark.kettenis@xs4all.nl> wrote:
>> From: Pedro Alves <pedro@codesourcery.com>
>> Date: Sun, 22 Mar 2009 16:47:09 +0000
>>
>> On Sunday 22 March 2009 16:43:36, Hui Zhu wrote:
>> > About this part, I think it again. ?I think keep it to macro is
>> > better. ?Because all of values can't be change. ?Set them to variable
>> > just to make record linux code more friendly to other arch. ?So I
>> > think make it to macro is more clear.
>>
>> I don't think it's clearer at all. ?I think it's just clutter.
>
> For what it is worth, I agree with Pedro here. ?If there is a bunch of
> constants like this that are only used once, adding #defines for them
> is a bit counterproductive.
>
> Oh, and in one of the comments there is a space missing in Linux2.4.
>
>
I think most of size between 2.4 and 2.6 are same because these
variables are interface to user space.

I will check them after p record checked-in.  If I find some of them
are not same, I will add a command can set it.
Do you think it's OK?

Thanks,
Hui


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