Re: [PING] Re: [PATCH v2 0/2] gdb/python: implement support for sending custom MI async notifications

["Jan Vraný" <Jan.Vrany@labware.com>]

Polite ping, 

Thanks! Jan


On Wed, 2023-09-27 at 19:56 +0100, Jan Vrany wrote:
> Polite ping.
> 
> Thanks! Jan
> 
> On Thu, 2023-09-21 at 11:50 +0100, Jan Vrany wrote:
> > Polite ping.
> > 
> > Thanks! Jan
> > 
> > On Wed, 2023-09-13 at 15:38 +0100, Jan Vrany wrote:
> > > I hope I addressed all comments raised by Andrew and Eli
> > > in this version. The main change is added restriction on
> > > NAME parameter and hopefully more clear wording in
> > > documentation.
> > > 
> > > Thanks, Jan
> > > 
> > > Changes since V1:
> > > 
> > > * Moved serialize_mi_result and helpers to py-mi.c as suggested.
> > >   Also, renamed serialize_mi_result to serialize_mi_results
> > >   instead of serialize_mi_data and do not rename serialize_mi_result_1.
> > > 
> > >   The reason for this is that GDB documentation,
> > >   section GDB/MI Output Syntax, describes "result" being single 'variable = value'
> > >   pair and "result-record" and "async-output" contains zero or more of these "result"s,
> > >   so calling top-level serialization function serialize_mi_results seems
> > >   a better name than serialize_mi_data (used in previous version).
> > > 
> > > * Made gdb.notify_mi DATA parameter optional as suggested.
> > > 
> > > * Validate gdb.notify_mi NAME parameter as suggested.
> > > 
> > > * Updated documentation:
> > >   * example formatted as black tool would do it,
> > >   * rephrase some sentences as suggested,
> > >   * document that DATA parameter is optional,
> > >   * document restrictions on NAME parameter and
> > >   * document that users should prefix user-defined notification
> > >     with hyphen to avoid possible conflicts.
> > > 
> > > * Add more tests.
> > > 
> > > 
> > > Jan Vrany (2):
> > >   gdb/python: generalize serialize_mi_result()
> > >   gdb/python: implement support for sending custom MI async
> > >     notifications
> > > 
> > >  gdb/NEWS                                  |   3 +
> > >  gdb/doc/python.texi                       |  45 +++++
> > >  gdb/python/py-mi.c                        | 230 ++++++++++++++++++++++
> > >  gdb/python/py-micmd.c                     | 185 +----------------
> > >  gdb/python/python-internal.h              |  18 ++
> > >  gdb/python/python.c                       |   4 +
> > >  gdb/testsuite/gdb.python/py-mi-notify.exp |  71 +++++++
> > >  7 files changed, 380 insertions(+), 176 deletions(-)
> > >  create mode 100644 gdb/testsuite/gdb.python/py-mi-notify.exp
> > > 
> > 
>