public inbox for gdb-testers@sourceware.org
help / color / mirror / Atom feed
* [binutils-gdb] Use gdb::function_view in iterate_over_symtabs & co
@ 2017-02-23 19:28 sergiodj+buildbot
2017-02-23 19:28 ` Failures on Ubuntu-AArch64-native-gdbserver-m64, branch master sergiodj+buildbot
` (8 more replies)
0 siblings, 9 replies; 10+ messages in thread
From: sergiodj+buildbot @ 2017-02-23 19:28 UTC (permalink / raw)
To: gdb-testers
*** TEST RESULTS FOR COMMIT 14bc53a81471e0b550de1c24d4d5266f676aacc3 ***
Author: Pedro Alves <palves@redhat.com>
Branch: master
Commit: 14bc53a81471e0b550de1c24d4d5266f676aacc3
Use gdb::function_view in iterate_over_symtabs & co
I wanted to pass a lambda to iterate_over_symtabs (see following
patch), so I converted it to function_view, and then the rest is
cascaded from that.
This gets rid of a bunch of single-use callback functions and
corresponding manually managed callback capture types
(add_partial_datum, search_symbols_data, etc.) in favor of letting the
compiler generate them for us by using lambdas with a capture. In a
couple cases, it was more natural to convert the existing function
callbacks to function objects (i.e., operator(), e.g.,
decode_compound_collector).
gdb/ChangeLog:
2017-02-23 Pedro Alves <palves@redhat.com>
* ada-lang.c: Include "common/function-view.h".
(ada_iterate_over_symbols): Adjust to use function_view as
callback type.
(struct add_partial_datum, ada_complete_symbol_matcher): Delete.
(ada_make_symbol_completion_list): Use a lambda.
(ada_exc_search_name_matches): Delete.
(name_matches_regex): New.
(ada_add_global_exceptions): Use a lambda and name_matches_regex.
* compile/compile-c-support.c: Include "common/function-view.h".
(print_one_macro): Change prototype to accept a ui_file pointer.
(write_macro_definitions): Use a lambda.
* dwarf2read.c: Include "common/function-view.h".
(dw2_map_expand_apply, dw2_map_symtabs_matching_filename)
(dw2_expand_symtabs_matching): Adjust to use function_view as
callback type.
* language.h: Include "common/function-view.h".
(struct language_defn) <la_iterate_over_symbols>: Adjust to use
function_view as callback type.
(LA_ITERATE_OVER_SYMBOLS): Remove DATA parameter.
* linespec.c: Include "common/function-view.h".
(collect_info::add_symbol): New method.
(struct symbol_and_data_callback, iterate_inline_only, struct
symbol_matcher_data, iterate_name_matcher): Delete.
(iterate_over_all_matching_symtabs): Adjust to use function_view
as callback type and lambdas.
(iterate_over_file_blocks): Adjust to use function_view as
callback type.
(decode_compound_collector): Now a class with private fields.
(decode_compound_collector::release_symbols): New method.
(collect_one_symbol): Rename to...
(decode_compound_collector::operator()): ... this and adjust.
(lookup_prefix_sym): decode_compound_collector construction bits
move to decode_compound_collector ctor. Pass the
decode_compound_collector object directly as callback. Remove
cleanups and use decode_compound_collector::release_symbols
instead.
(symtab_collector): Now a class with private fields.
(symtab_collector::release_symtabs): New method.
(add_symtabs_to_list): Rename to...
(symtab_collector::operator()): ... this and adjust.
(collect_symtabs_from_filename): symtab_collector construction
bits move to symtab_collector ctor. Pass the symtab_collector
object directly as callback. Remove cleanups and use
symtab_collector::release_symtabs instead.
(collect_symbols): Delete.
(add_matching_symbols_to_info): Use lambdas.
* macrocmd.c (print_macro_callback): Delete.
(info_macro_command): Use a lambda.
(info_macros_command): Pass print_macro_definition as callable
directly.
(print_one_macro): Remove 'ignore' parameter.
(macro_list_command): Adjust.
* macrotab.c (macro_for_each_data::fn): Now a function_view.
(macro_for_each_data::user_data): Delete field.
(foreach_macro): Adjust to call the function_view.
(macro_for_each): Adjust to use function_view as callback type.
(foreach_macro_in_scope): Adjust to call the function_view.
(macro_for_each_in_scope): Adjust to use function_view as callback
type.
* macrotab.h: Include "common/function-view.h".
(macro_callback_fn): Declare a prototype instead of a pointer.
Remove "user_data" parameter.
(macro_for_each, macro_for_each_in_scope): Adjust to use
function_view as callback type.
* psymtab.c (partial_map_expand_apply)
(psym_map_symtabs_matching_filename, recursively_search_psymtabs):
Adjust to use function_view as callback type and to return bool.
(psym_expand_symtabs_matching): Adjust to use function_view as
callback types.
* symfile-debug.c (debug_qf_map_symtabs_matching_filename): Adjust
to use function_view as callback type and to return bool.
(debug_qf_expand_symtabs_matching): Adjust to use function_view as
callback types.
* symfile.c (expand_symtabs_matching): Adjust to use function_view
as callback types.
* symfile.h: Include "common/function-view.h".
(expand_symtabs_file_matcher_ftype)
(expand_symtabs_symbol_matcher_ftype)
(expand_symtabs_exp_notify_ftype): Remove "data" parameter and
return bool.
(quick_symbol_functions::map_symtabs_matching_filename)
(quick_symbol_functions::expand_symtabs_matching): Adjust to use
function_view as callback type and return bool.
(expand_symtabs_matching): Adjust to use function_view as callback
type.
(maintenance_expand_name_matcher)
(maintenance_expand_file_matcher): Delete.
(maintenance_expand_symtabs): Use lambdas.
* symtab.c (iterate_over_some_symtabs): Adjust to use
function_view as callback types and return bool.
(iterate_over_symtabs): Likewise. Use unique_xmalloc_ptr instead
of a cleanup.
(lookup_symtab_callback): Delete.
(lookup_symtab): Use a lambda.
(iterate_over_symbols): Adjust to use function_view as callback
type.
(struct search_symbols_data, search_symbols_file_matches)
(search_symbols_name_matches): Delete.
(search_symbols): Use a pair of lambdas.
(struct add_name_data, add_macro_name, symbol_completion_matcher)
(symtab_expansion_callback): Delete.
(default_make_symbol_completion_list_break_on_1): Use lambdas.
* symtab.h: Include "common/function-view.h".
(iterate_over_some_symtabs): Adjust to use function_view as
callback type and return bool.
(iterate_over_symtabs): Adjust to use function_view as callback
type.
(symbol_found_callback_ftype): Remove 'data' parameter and return
bool.
(iterate_over_symbols): Adjust to use function_view as callback
type.
^ permalink raw reply [flat|nested] 10+ messages in thread
* Failures on Ubuntu-AArch64-native-gdbserver-m64, branch master
2017-02-23 19:28 [binutils-gdb] Use gdb::function_view in iterate_over_symtabs & co sergiodj+buildbot
@ 2017-02-23 19:28 ` sergiodj+buildbot
2017-02-23 19:56 ` Failures on Debian-s390x-native-gdbserver-m64, " sergiodj+buildbot
` (7 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: sergiodj+buildbot @ 2017-02-23 19:28 UTC (permalink / raw)
To: gdb-testers
Buildslave:
ubuntu-aarch64-1
Full Build URL:
<http://gdb-build.sergiodj.net/builders/Ubuntu-AArch64-native-gdbserver-m64/builds/1469>
Commit(s) tested:
14bc53a81471e0b550de1c24d4d5266f676aacc3
Author(s) (in the same order as the commits):
Pedro Alves <palves@redhat.com>
Subject:
Use gdb::function_view in iterate_over_symtabs & co
Testsuite log (gdb.sum and gdb.log) URL(s):
<http://gdb-build.sergiodj.net/results/Ubuntu-AArch64-native-gdbserver-m64/14/14bc53a81471e0b550de1c24d4d5266f676aacc3/>
*** Diff to previous build ***
============================
PASS -> FAIL: gdb.trace/mi-tsv-changed.exp: create delete modify: tvariable $tvar3 modified
============================
*** Complete list of XFAILs for this builder ***
To obtain the list of XFAIL tests for this builder, go to:
<http://git.sergiodj.net/?p=gdb-xfails.git;a=blob;f=xfails/Ubuntu-AArch64-native-gdbserver-m64/xfails/master/xfail;hb=84d1683>
You can also see a pretty-printed version of the list, with more information
about each XFAIL, by going to:
<http://git.sergiodj.net/?p=gdb-xfails.git;a=blob;f=xfails/Ubuntu-AArch64-native-gdbserver-m64/xfails/master/xfail.table;hb=84d1683>
^ permalink raw reply [flat|nested] 10+ messages in thread
* Failures on Debian-s390x-native-gdbserver-m64, branch master
2017-02-23 19:28 [binutils-gdb] Use gdb::function_view in iterate_over_symtabs & co sergiodj+buildbot
2017-02-23 19:28 ` Failures on Ubuntu-AArch64-native-gdbserver-m64, branch master sergiodj+buildbot
@ 2017-02-23 19:56 ` sergiodj+buildbot
2017-02-23 20:15 ` Failures on Fedora-x86_64-native-extended-gdbserver-m64, " sergiodj+buildbot
` (6 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: sergiodj+buildbot @ 2017-02-23 19:56 UTC (permalink / raw)
To: gdb-testers
Buildslave:
debian-jessie-s390x-1
Full Build URL:
<http://gdb-build.sergiodj.net/builders/Debian-s390x-native-gdbserver-m64/builds/3912>
Commit(s) tested:
14bc53a81471e0b550de1c24d4d5266f676aacc3
Author(s) (in the same order as the commits):
Pedro Alves <palves@redhat.com>
Subject:
Use gdb::function_view in iterate_over_symtabs & co
Testsuite log (gdb.sum and gdb.log) URL(s):
<http://gdb-build.sergiodj.net/results/Debian-s390x-native-gdbserver-m64/14/14bc53a81471e0b550de1c24d4d5266f676aacc3/>
^ permalink raw reply [flat|nested] 10+ messages in thread
* Failures on Ubuntu-AArch32-native-extended-gdbserver-m32, branch master
2017-02-23 19:28 [binutils-gdb] Use gdb::function_view in iterate_over_symtabs & co sergiodj+buildbot
` (2 preceding siblings ...)
2017-02-23 20:15 ` Failures on Fedora-x86_64-native-extended-gdbserver-m64, " sergiodj+buildbot
@ 2017-02-23 20:15 ` sergiodj+buildbot
2017-02-23 20:34 ` Failures on Ubuntu-AArch32-m32, " sergiodj+buildbot
` (4 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: sergiodj+buildbot @ 2017-02-23 20:15 UTC (permalink / raw)
To: gdb-testers
Buildslave:
ubuntu-trusty-aarch32-1
Full Build URL:
<http://gdb-build.sergiodj.net/builders/Ubuntu-AArch32-native-extended-gdbserver-m32/builds/94>
Commit(s) tested:
14bc53a81471e0b550de1c24d4d5266f676aacc3
Author(s) (in the same order as the commits):
Pedro Alves <palves@redhat.com>
Subject:
Use gdb::function_view in iterate_over_symtabs & co
Testsuite log (gdb.sum and gdb.log) URL(s):
<http://gdb-build.sergiodj.net/results/Ubuntu-AArch32-native-extended-gdbserver-m32/14/14bc53a81471e0b550de1c24d4d5266f676aacc3/>
*** Diff to previous build ***
============================
PASS -> FAIL: gdb.threads/pthreads.exp: check backtrace from thread 1
============================
*** Complete list of XFAILs for this builder ***
FAILURE TO OBTAIN THE COMMIT FOR THE XFAIL LIST. PLEASE CONTACT THE BUILDBOT ADMIN.
^ permalink raw reply [flat|nested] 10+ messages in thread
* Failures on Fedora-x86_64-native-extended-gdbserver-m64, branch master
2017-02-23 19:28 [binutils-gdb] Use gdb::function_view in iterate_over_symtabs & co sergiodj+buildbot
2017-02-23 19:28 ` Failures on Ubuntu-AArch64-native-gdbserver-m64, branch master sergiodj+buildbot
2017-02-23 19:56 ` Failures on Debian-s390x-native-gdbserver-m64, " sergiodj+buildbot
@ 2017-02-23 20:15 ` sergiodj+buildbot
2017-02-23 20:15 ` Failures on Ubuntu-AArch32-native-extended-gdbserver-m32, " sergiodj+buildbot
` (5 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: sergiodj+buildbot @ 2017-02-23 20:15 UTC (permalink / raw)
To: gdb-testers
Buildslave:
fedora-x86-64-4
Full Build URL:
<http://gdb-build.sergiodj.net/builders/Fedora-x86_64-native-extended-gdbserver-m64/builds/5523>
Commit(s) tested:
14bc53a81471e0b550de1c24d4d5266f676aacc3
Author(s) (in the same order as the commits):
Pedro Alves <palves@redhat.com>
Subject:
Use gdb::function_view in iterate_over_symtabs & co
Testsuite log (gdb.sum and gdb.log) URL(s):
<http://gdb-build.sergiodj.net/results/Fedora-x86_64-native-extended-gdbserver-m64/14/14bc53a81471e0b550de1c24d4d5266f676aacc3/>
*** Diff to previous build ***
============================
PASS -> FAIL: gdb.base/new-ui.exp: do_test: delete all breakpoints on extra console
PASS -> FAIL: gdb.base/new-ui.exp: do_test: main console: next causes no spurious output on other console
============================
*** Complete list of XFAILs for this builder ***
To obtain the list of XFAIL tests for this builder, go to:
<http://git.sergiodj.net/?p=gdb-xfails.git;a=blob;f=xfails/Fedora-x86_64-native-extended-gdbserver-m64/xfails/master/xfail;hb=4eb872a>
You can also see a pretty-printed version of the list, with more information
about each XFAIL, by going to:
<http://git.sergiodj.net/?p=gdb-xfails.git;a=blob;f=xfails/Fedora-x86_64-native-extended-gdbserver-m64/xfails/master/xfail.table;hb=4eb872a>
^ permalink raw reply [flat|nested] 10+ messages in thread
* Failures on Ubuntu-AArch32-m32, branch master
2017-02-23 19:28 [binutils-gdb] Use gdb::function_view in iterate_over_symtabs & co sergiodj+buildbot
` (3 preceding siblings ...)
2017-02-23 20:15 ` Failures on Ubuntu-AArch32-native-extended-gdbserver-m32, " sergiodj+buildbot
@ 2017-02-23 20:34 ` sergiodj+buildbot
2017-02-24 20:21 ` Failures on Fedora-ppc64be-native-gdbserver-m64, " sergiodj+buildbot
` (3 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: sergiodj+buildbot @ 2017-02-23 20:34 UTC (permalink / raw)
To: gdb-testers
Buildslave:
ubuntu-trusty-aarch32-1
Full Build URL:
<http://gdb-build.sergiodj.net/builders/Ubuntu-AArch32-m32/builds/94>
Commit(s) tested:
14bc53a81471e0b550de1c24d4d5266f676aacc3
Author(s) (in the same order as the commits):
Pedro Alves <palves@redhat.com>
Subject:
Use gdb::function_view in iterate_over_symtabs & co
Testsuite log (gdb.sum and gdb.log) URL(s):
<http://gdb-build.sergiodj.net/results/Ubuntu-AArch32-m32/14/14bc53a81471e0b550de1c24d4d5266f676aacc3/>
*** Diff to previous build ***
============================
PASS -> FAIL: gdb.threads/pthreads.exp: check backtrace from thread 1
============================
*** Complete list of XFAILs for this builder ***
FAILURE TO OBTAIN THE COMMIT FOR THE XFAIL LIST. PLEASE CONTACT THE BUILDBOT ADMIN.
^ permalink raw reply [flat|nested] 10+ messages in thread
* Failures on Fedora-ppc64be-native-gdbserver-m64, branch master
2017-02-23 19:28 [binutils-gdb] Use gdb::function_view in iterate_over_symtabs & co sergiodj+buildbot
` (4 preceding siblings ...)
2017-02-23 20:34 ` Failures on Ubuntu-AArch32-m32, " sergiodj+buildbot
@ 2017-02-24 20:21 ` sergiodj+buildbot
2017-02-24 21:29 ` Failures on Fedora-ppc64be-native-extended-gdbserver-m64, " sergiodj+buildbot
` (2 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: sergiodj+buildbot @ 2017-02-24 20:21 UTC (permalink / raw)
To: gdb-testers
Buildslave:
fedora-ppc64be-1
Full Build URL:
<http://gdb-build.sergiodj.net/builders/Fedora-ppc64be-native-gdbserver-m64/builds/5106>
Commit(s) tested:
14bc53a81471e0b550de1c24d4d5266f676aacc3
Author(s) (in the same order as the commits):
Pedro Alves <palves@redhat.com>
Subject:
Use gdb::function_view in iterate_over_symtabs & co
Testsuite log (gdb.sum and gdb.log) URL(s):
<http://gdb-build.sergiodj.net/results/Fedora-ppc64be-native-gdbserver-m64/14/14bc53a81471e0b550de1c24d4d5266f676aacc3/>
^ permalink raw reply [flat|nested] 10+ messages in thread
* Failures on Fedora-ppc64be-native-extended-gdbserver-m64, branch master
2017-02-23 19:28 [binutils-gdb] Use gdb::function_view in iterate_over_symtabs & co sergiodj+buildbot
` (5 preceding siblings ...)
2017-02-24 20:21 ` Failures on Fedora-ppc64be-native-gdbserver-m64, " sergiodj+buildbot
@ 2017-02-24 21:29 ` sergiodj+buildbot
2017-02-25 3:42 ` Failures on Fedora-ppc64le-native-gdbserver-m64, " sergiodj+buildbot
2017-02-25 5:48 ` Failures on Fedora-ppc64le-native-extended-gdbserver-m64, " sergiodj+buildbot
8 siblings, 0 replies; 10+ messages in thread
From: sergiodj+buildbot @ 2017-02-24 21:29 UTC (permalink / raw)
To: gdb-testers
Buildslave:
fedora-ppc64be-1
Full Build URL:
<http://gdb-build.sergiodj.net/builders/Fedora-ppc64be-native-extended-gdbserver-m64/builds/5122>
Commit(s) tested:
14bc53a81471e0b550de1c24d4d5266f676aacc3
Author(s) (in the same order as the commits):
Pedro Alves <palves@redhat.com>
Subject:
Use gdb::function_view in iterate_over_symtabs & co
Testsuite log (gdb.sum and gdb.log) URL(s):
<http://gdb-build.sergiodj.net/results/Fedora-ppc64be-native-extended-gdbserver-m64/14/14bc53a81471e0b550de1c24d4d5266f676aacc3/>
^ permalink raw reply [flat|nested] 10+ messages in thread
* Failures on Fedora-ppc64le-native-gdbserver-m64, branch master
2017-02-23 19:28 [binutils-gdb] Use gdb::function_view in iterate_over_symtabs & co sergiodj+buildbot
` (6 preceding siblings ...)
2017-02-24 21:29 ` Failures on Fedora-ppc64be-native-extended-gdbserver-m64, " sergiodj+buildbot
@ 2017-02-25 3:42 ` sergiodj+buildbot
2017-02-25 5:48 ` Failures on Fedora-ppc64le-native-extended-gdbserver-m64, " sergiodj+buildbot
8 siblings, 0 replies; 10+ messages in thread
From: sergiodj+buildbot @ 2017-02-25 3:42 UTC (permalink / raw)
To: gdb-testers
Buildslave:
fedora-ppc64le-1
Full Build URL:
<http://gdb-build.sergiodj.net/builders/Fedora-ppc64le-native-gdbserver-m64/builds/5037>
Commit(s) tested:
14bc53a81471e0b550de1c24d4d5266f676aacc3
Author(s) (in the same order as the commits):
Pedro Alves <palves@redhat.com>
Subject:
Use gdb::function_view in iterate_over_symtabs & co
Testsuite log (gdb.sum and gdb.log) URL(s):
<http://gdb-build.sergiodj.net/results/Fedora-ppc64le-native-gdbserver-m64/14/14bc53a81471e0b550de1c24d4d5266f676aacc3/>
^ permalink raw reply [flat|nested] 10+ messages in thread
* Failures on Fedora-ppc64le-native-extended-gdbserver-m64, branch master
2017-02-23 19:28 [binutils-gdb] Use gdb::function_view in iterate_over_symtabs & co sergiodj+buildbot
` (7 preceding siblings ...)
2017-02-25 3:42 ` Failures on Fedora-ppc64le-native-gdbserver-m64, " sergiodj+buildbot
@ 2017-02-25 5:48 ` sergiodj+buildbot
8 siblings, 0 replies; 10+ messages in thread
From: sergiodj+buildbot @ 2017-02-25 5:48 UTC (permalink / raw)
To: gdb-testers
Buildslave:
fedora-ppc64le-1
Full Build URL:
<http://gdb-build.sergiodj.net/builders/Fedora-ppc64le-native-extended-gdbserver-m64/builds/5052>
Commit(s) tested:
14bc53a81471e0b550de1c24d4d5266f676aacc3
Author(s) (in the same order as the commits):
Pedro Alves <palves@redhat.com>
Subject:
Use gdb::function_view in iterate_over_symtabs & co
Testsuite log (gdb.sum and gdb.log) URL(s):
<http://gdb-build.sergiodj.net/results/Fedora-ppc64le-native-extended-gdbserver-m64/14/14bc53a81471e0b550de1c24d4d5266f676aacc3/>
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2017-02-25 5:48 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-02-23 19:28 [binutils-gdb] Use gdb::function_view in iterate_over_symtabs & co sergiodj+buildbot
2017-02-23 19:28 ` Failures on Ubuntu-AArch64-native-gdbserver-m64, branch master sergiodj+buildbot
2017-02-23 19:56 ` Failures on Debian-s390x-native-gdbserver-m64, " sergiodj+buildbot
2017-02-23 20:15 ` Failures on Fedora-x86_64-native-extended-gdbserver-m64, " sergiodj+buildbot
2017-02-23 20:15 ` Failures on Ubuntu-AArch32-native-extended-gdbserver-m32, " sergiodj+buildbot
2017-02-23 20:34 ` Failures on Ubuntu-AArch32-m32, " sergiodj+buildbot
2017-02-24 20:21 ` Failures on Fedora-ppc64be-native-gdbserver-m64, " sergiodj+buildbot
2017-02-24 21:29 ` Failures on Fedora-ppc64be-native-extended-gdbserver-m64, " sergiodj+buildbot
2017-02-25 3:42 ` Failures on Fedora-ppc64le-native-gdbserver-m64, " sergiodj+buildbot
2017-02-25 5:48 ` Failures on Fedora-ppc64le-native-extended-gdbserver-m64, " sergiodj+buildbot
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).