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: [PREC]RFA: Add the 'options' to record_beneath_to_wait


On Fri, Aug 21, 2009 at 11:49, Michael Snyder<msnyder@vmware.com> wrote:
> Hui Zhu wrote:
>>
>> Hi,
>>
>> I found that there is a new argument of to_wait -- options. ?But in
>> record_wait, it isn't put to record_beneath_to_wait.
>> So I make a patch for it. ?And I test it with testsuit in ubuntu i386 is
>> OK.
>>
>> Please help me review it.
>
> Looks good, please check it in.
> Thanks!

Checked in.

Thanks,
Hui

>
>> prec-fix-record_wait_options.txt
>> 2009-08-21 ?Hui Zhu ?<teawater@gmail.com>
>>
>> ? ? ? ?* record.c (record_wait): Change the argument of
>> ? ? ? ?record_beneath_to_wait to options.
>>
>> ---
>> ?record.c | ? ?4 ++--
>> ?1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> --- a/record.c
>> +++ b/record.c
>> @@ -598,7 +598,7 @@ record_wait (struct target_ops *ops,
>> ? ? ? ?{
>> ? ? ? ? ?/* This is a single step. ?*/
>> ? ? ? ? ?return record_beneath_to_wait (record_beneath_to_wait_ops,
>> - ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?ptid, status, 0);
>> + ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?ptid, status, options);
>> ? ? ? ?}
>> ? ? ? else
>> ? ? ? ?{
>> @@ -609,7 +609,7 @@ record_wait (struct target_ops *ops,
>> ? ? ? ? ?while (1)
>> ? ? ? ? ? ?{
>> ? ? ? ? ? ? ?ret = record_beneath_to_wait (record_beneath_to_wait_ops,
>> - ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ptid, status, 0);
>> + ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ptid, status, options);
>>
>> ? ? ? ? ? ? ?if (status->kind == TARGET_WAITKIND_STOPPED
>> ? ? ? ? ? ? ? ? ?&& status->value.sig == TARGET_SIGNAL_TRAP)
>
>


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