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: [RFC] Autoload-breakpoints new version [3/9] notification async


On Thu, Aug 23, 2012 at 8:46 PM, Yao Qi <yao@codesourcery.com> wrote:
> On 08/15/2012 10:22 PM, Hui Zhu wrote:
>>
>>   struct stop_reply;
>> @@ -1297,6 +1302,7 @@ enum {
>>     PACKET_QDisableRandomization,
>>     PACKET_QAgent,
>>     PACKET_AutoloadBreakpoints,
>> +  PACKET_NotificationAsync,
>>     PACKET_MAX
>>   };
>>
>> @@ -3998,6 +4004,8 @@ static struct protocol_feature remote_pr
>>       remote_string_tracing_feature, -1 },
>>     { "AutoloadBreakpoints", PACKET_DISABLE, remote_supported_packet,
>>       PACKET_AutoloadBreakpoints },
>> +  { "NotificationAsync", PACKET_DISABLE, remote_supported_packet,
>> +    PACKET_NotificationAsync },
>>   };
>
>
> Hui,
> As we discussed in this thread,
>
>   Re: [RFC] Autoload-breakpoints new version [3/9] notification async
>   http://sourceware.org/ml/gdb-patches/2012-08/msg00337.html
>
> "NotificationAsync" is not needed anymore, so your patch should be updated
> to reflect this.
>
> --
> Yao

Hi Yao,

1. We still not sure keep the support packet "NotificationAsync" is
good or not.
2. Even if we don't need support packet "NotificationAsync".  The
notification still need the extend for async.

So if you don't mind, I suggest you keep review this patch.  And after
I got your comments, I update a new one.

Thanks,
Hui


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