Bob Rossi wrote: > On Tue, Jan 24, 2006 at 07:32:15PM +0300, Vladimir Prus wrote: >> >> Hello! >> >> The attached patch adds the "fullname" field to the output of MI >> -break-info command. > > The patch seems good to me at first glance. However, you should also > update the testsuite, and send in the patch for that as well. > > Also, don't forget about the documentation. Revised patch attached. I did verify that it causes no regressions for MI tests. Note, however, that I could not verify that it does not cause any regressions globally, because something seems wrong with my configuration or the test system. Sporadically, I get errors like this: Running ../.././gdb/testsuite/gdb.base/code-expr.exp ... gdb compile failed, spawn failed WARNING: Testcase compile failed, so all tests in this file will automatically fail. WARNING: remote_expect statement without a default case?! ERROR: couldn't load /space/p2/ghost/build/gdb/gdb/testsuite/gdb.base/cvexpr into /space/p2/ghost/build/gdb/gdb/testsuite/../../gdb/gdb (end of file). FAIL: gdb.base/code-expr.exp: set print address off Or messages like: gdb compile failed, gcc: fork: Resource temporarily unavailable When running specific tests, the errors don't reproduce. Sometimes the errors disappear by themself. So, it looks like testsystem needs more resources (pids?) than is allowed by rlimit or something like that. - Volodya