[PATCH 4/4] [gdb/testsuite] Speed up gdb.python/py-rbreak.exp

[Tom de Vries <tdevries@suse.de>]

I noticed that gdb.python/py-rbreak.exp does:
...
if {![runto_main]} {
    return
}

gdb_test_no_output "nosharedlibrary"
...

Speed up the test-case by not running to main.

This breaks the gdb.lookup_symbol("efunc1") call, which requires a current
frame, so use lookup_global_symbol instead.

Also remove the nosharedlibrary command.  No longer running to main means we
no longer load symbols for shared libraries, so the nosharedlibrary that drops
those symbols is no longer required.
---
 gdb/testsuite/gdb.python/py-rbreak.exp | 10 ++--------
 1 file changed, 2 insertions(+), 8 deletions(-)

diff --git a/gdb/testsuite/gdb.python/py-rbreak.exp b/gdb/testsuite/gdb.python/py-rbreak.exp
index 395a603ef4e..a1ca7d4283d 100644
--- a/gdb/testsuite/gdb.python/py-rbreak.exp
+++ b/gdb/testsuite/gdb.python/py-rbreak.exp
@@ -26,12 +26,6 @@ if {[prepare_for_testing "failed to prepare" ${testfile} [list $srcfile $srcfile
     return 1
 }
 
-if {![runto_main]} {
-    return
-}
-
-gdb_test_no_output "nosharedlibrary"
-
 gdb_py_test_silent_cmd {py sl = gdb.rbreak("",minsyms=False)} \
     "get all function breakpoints" 0
 set min_breakpoints 11
@@ -65,9 +59,9 @@ gdb_py_test_silent_cmd {py sl = gdb.rbreak("func1",minsyms=True)} \
 gdb_test "py print(len(sl))" "2" \
     "check number of returned breakpoints is 2"
 
-gdb_py_test_silent_cmd {python sym = gdb.lookup_symbol("efunc1")} \
+gdb_py_test_silent_cmd {python sym = gdb.lookup_global_symbol("efunc1")} \
     "find a symbol in objfile" 1
-gdb_py_test_silent_cmd {python symtab = sym[0].symtab} \
+gdb_py_test_silent_cmd {python symtab = sym.symtab} \
     "get backing symbol table" 1
 gdb_py_test_silent_cmd \
     [subst_vars \
-- 
2.51.0