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: [python] [patch] Fix PR/12183


>>>>> "Phil" == Phil Muldoon <pmuldoon@redhat.com> writes:

Phil> This patch fixes PR python/12183.  In this case GDB was printing a
Phil> stack trace for a gdb.GdbError.  This is incorrect, GdbErrors
Phil> should not produce a stack trace.  This fix is largely a duplicate
Phil> of the code already in py-cmd.

Phil> OK?

Looks good, just one nit.

Phil> +  if (! result)

We decided not to put a space after '! 'here.

Phil> +      if (! PyErr_GivenExceptionMatches (ptype, gdbpy_gdberror_exc)

And likewise.

Ok with this fixed.

Tom


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