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] gdb.ada/catch_ex.exp, gdb.ada/mi_catch_ex.exp and unsupported catchpoints


> (gdb) catch exception
> Cannot insert catchpoints in this configuration.

We're going to have to talk to the guys who decided to build the Ada
runtime without debugging info. The normal way to build the runtime
is no debug info *except* for a few files. This is really doing a
disservice to the users!

Grumble, grumble, grumble. Sorry.

> but they don't match this specific error message.  I can't seem to find the
> "Cannot break on" string anywhere, is it escaping me?

It's actuall there, see ada-lang.c:ada_exception_sal. But upon more
careful inspection, I think it might have become dead code:
The error is only raised if we failed to look that symbol up, but
the call to ada_exception_support_info_sniffer to have already errored
out if that was the case. So I think we can replace that by an assertion
and simplify the testcase.

That's going to be on me, but I won't have much time this week.

In the meantime...

> (BTW, the "cannot insert catchpoints" error may be a bit confusing,
> considering there are other non-Ada catchpoints that will work just
> fine.)

I'll try to think of a better error message - it's not my forte...

> gdb/testsuite/
> 2011-12-06  Pedro Alves  <pedro@codesourcery.com>
> 
> 	* gdb.ada/catch_ex.exp: Skip as unsupported if "catch exception"
> 	throws "Cannot insert catchpoints in this configuration".
> 	* gdb.ada/mi_catch_ex.exp: Likewise.

... I think that this is a good stop-gap measure. It helps you, so
it's OK to go in. I'll cleanup afterwards anyways.  Thanks for doing
that.

-- 
Joel


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