[PATCH v2 16/22] Rewrite TRY/CATCH

Pedro Alves palves@redhat.com
Wed Apr 3 17:03:00 GMT 2019


On 02/27/2019 08:18 PM, Tom Tromey wrote:
> This rewrites gdb's TRY/CATCH to plain C++ try/catch.  The patch was
> largely written by script, though one change (to a comment in
> common-exceptions.h) was reverted by hand.

I reviewed this one and the next as a squashed view (any reason you
didn't squash them?), and it looks OK to me, though personally
I'd much prefer to see the explicit "struct" in:

 -  CATCH (e, RETURN_MASK_ERROR)
 +  catch (const struct gdb_exception_error &e)

go away, and end up with just:

   catch (const gdb_exception_error &e)

Thanks,
Pedro Alves



More information about the Gdb-patches mailing list