Thanks for the feedback, I attached an additional patch, containing the requested changes in NEWS and gdb.texinfo. Please, consider revising it, and forgive me in advance for mistreating the English grammar! Regards, Giuseppe > -----Original Message----- > From: Tom Tromey [mailto:tromey@redhat.com] > Sent: Wednesday, October 17, 2012 10:59 PM > To: Giuseppe MONTALTO > Cc: gdb-patches@sourceware.org; Abid, Hafiz > Subject: Re: [PATCH] enhancement of mi_cmd_data_write_memory_bytes > for filling memory regions (was [PATCH] new MI command for pattern filling > of memory regions) > > >>>>> "Giuseppe" == Giuseppe MONTALTO > writes: > > Sorry about the delay on this. > > Giuseppe> New patch attached: > Giuseppe> - removed trivial comments > Giuseppe> - removed redundant "else if" clause Any further feedback is > Giuseppe> always welcome, > > The patch looks good. > Is there a documentation part as well? I forget. > If so, and it has been approved, it should all go in with a single commit. > If not, then you need to patch gdb.texinfo and NEWS as well. > > thanks, > Tom