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: [PATCH 0/5] Make gdbserver use exceptions and cleanups


On 08/18/2014 09:46 AM, Gary Benson wrote:

> gdbserver has the rudiments of an exception system, but it's kind of
> hacky.  This series converts gdbserver to use the same exceptions and
> cleanups as GDB.
> 
> This series is fairly minimal in that only the catch/cleanup code in
> what was "main" has been updated to use the new system.  Other parts
> of gdbserver could likely be converted to TRY_CATCH and cleanups too,
> but that can be done on an as-needed basis if this patch is accepted.
> 
> A side-effect of this patch is that some error messages will change
> slightly, and in cases with multiple errors the error messages will
> be printed in a different order.
> 
> Built and regtested on RHEL 6.5 x86_64.
> 
> Ok to commit?

Other than the minor nits I just picked in the responses to the
patches, this looks great to me.

Thanks a lot for doing this.

Pedro Alves


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