From a8b3b8e93e36d2c9efe346f04f3bbd767e3cc0f0 Mon Sep 17 00:00:00 2001 From: Andrew Burgess Date: Wed, 2 Oct 2019 22:01:46 +0100 Subject: [PATCH] gdb: Remove whitespace in 'std::vector <...>' In the following 3 commits: commit df07e2c772dab40d268dc44c78bb087c4b75b3c6 Date: Wed Sep 25 16:10:50 2019 +0100 gdb: Remove a use of VEC from dwarf2read.{c,h} commit 554ac434b02465f1fc925b0ae3393fb841e0d59c Date: Thu Sep 19 13:17:59 2019 -0400 gdb: Change a VEC to std::vector in btrace.{c,h} commit 46f29a9a260da1a03176682aff63bad03d8f2e8b Date: Mon Sep 16 09:12:27 2019 -0400 gdb: Remove a VEC from gdbsupport/btrace-common.h I incorrectly wrote 'std::vector <...>' instead of 'std::vector<...>', this commit fixes this mistake. There should be no user visible changes after this commit. gdb/ChangeLog: * btrace.c (btrace_add_pc): Remove whitespace before the template parameter in 'std::vector <...>'. (parse_xml_btrace_block): Likewise. (btrace_maint_decode_pt): Likewise. (btrace_maint_update_packets): Likewise. (btrace_maint_print_packets): Likewise. * btrace.h (struct btrace_maint_info): Likewise. * dwarf2read.c (struct type_unit_group): Likewise. (build_type_psymtabs_reader): Likewise. * gdbsupport/btrace-common.c (btrace_data_append): Likewise. * gdbsupport/btrace-common.h (struct btrace_data_bts): Likewise. * nat/linux-btrace.c (perf_event_read_bts): Likewise. --- gdb/ChangeLog | 15 +++++++++++++++ gdb/btrace.c | 12 ++++++------ gdb/btrace.h | 2 +- gdb/dwarf2read.c | 4 ++-- gdb/gdbsupport/btrace-common.c | 2 +- gdb/gdbsupport/btrace-common.h | 2 +- gdb/nat/linux-btrace.c | 4 ++-- 7 files changed, 28 insertions(+), 13 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 0dabd9e91ca..bcf1de9e7ae 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,18 @@ +2019-10-03 Andrew Burgess + + * btrace.c (btrace_add_pc): Remove whitespace before the template + parameter in 'std::vector <...>'. + (parse_xml_btrace_block): Likewise. + (btrace_maint_decode_pt): Likewise. + (btrace_maint_update_packets): Likewise. + (btrace_maint_print_packets): Likewise. + * btrace.h (struct btrace_maint_info): Likewise. + * dwarf2read.c (struct type_unit_group): Likewise. + (build_type_psymtabs_reader): Likewise. + * gdbsupport/btrace-common.c (btrace_data_append): Likewise. + * gdbsupport/btrace-common.h (struct btrace_data_bts): Likewise. + * nat/linux-btrace.c (perf_event_read_bts): Likewise. + 2019-10-03 Tom de Vries * cli/cli-style.c (_initialize_cli_style): Adding a '.' at the end of diff --git a/gdb/btrace.c b/gdb/btrace.c index 8bed31cdac0..e2443a2d234 100644 --- a/gdb/btrace.c +++ b/gdb/btrace.c @@ -1579,7 +1579,7 @@ btrace_add_pc (struct thread_info *tp) pc = regcache_read_pc (regcache); btrace.format = BTRACE_FORMAT_BTS; - btrace.variant.bts.blocks = new std::vector ; + btrace.variant.bts.blocks = new std::vector; btrace.variant.bts.blocks->emplace_back (pc, pc); @@ -2036,7 +2036,7 @@ parse_xml_btrace_block (struct gdb_xml_parser *parser, case BTRACE_FORMAT_NONE: btrace->format = BTRACE_FORMAT_BTS; - btrace->variant.bts.blocks = new std::vector ; + btrace->variant.bts.blocks = new std::vector; break; default: @@ -2963,7 +2963,7 @@ btrace_maint_decode_pt (struct btrace_maint_info *maint, int errcode; if (maint->variant.pt.packets == NULL) - maint->variant.pt.packets = new std::vector ; + maint->variant.pt.packets = new std::vector; for (;;) { @@ -3095,7 +3095,7 @@ btrace_maint_update_packets (struct btrace_thread_info *btinfo, #if defined (HAVE_LIBIPT) case BTRACE_FORMAT_PT: if (btinfo->maint.variant.pt.packets == nullptr) - btinfo->maint.variant.pt.packets = new std::vector ; + btinfo->maint.variant.pt.packets = new std::vector; if (btinfo->maint.variant.pt.packets->empty ()) btrace_maint_update_pt_packets (btinfo); @@ -3123,7 +3123,7 @@ btrace_maint_print_packets (struct btrace_thread_info *btinfo, case BTRACE_FORMAT_BTS: { - const std::vector &blocks + const std::vector &blocks = *btinfo->data.variant.bts.blocks; unsigned int blk; @@ -3144,7 +3144,7 @@ btrace_maint_print_packets (struct btrace_thread_info *btinfo, #if defined (HAVE_LIBIPT) case BTRACE_FORMAT_PT: { - const std::vector &packets + const std::vector &packets = *btinfo->maint.variant.pt.packets; unsigned int pkt; diff --git a/gdb/btrace.h b/gdb/btrace.h index 208c089fa7c..acde64a5122 100644 --- a/gdb/btrace.h +++ b/gdb/btrace.h @@ -297,7 +297,7 @@ struct btrace_maint_info struct { /* A vector of decoded packets. */ - std::vector *packets; + std::vector *packets; /* The packet history iterator. We are iterating over the above PACKETS vector. */ diff --git a/gdb/dwarf2read.c b/gdb/dwarf2read.c index feac40ff95c..9d9dd6db709 100644 --- a/gdb/dwarf2read.c +++ b/gdb/dwarf2read.c @@ -620,7 +620,7 @@ struct type_unit_group /* The TUs that share this DW_AT_stmt_list entry. This is added to while parsing type units to build partial symtabs, and is deleted afterwards and not used again. */ - std::vector *tus; + std::vector *tus; /* The compunit symtab. Type units in a group needn't all be defined in the same source file, @@ -8191,7 +8191,7 @@ build_type_psymtabs_reader (const struct die_reader_specs *reader, tu_group = get_type_unit_group (cu, attr); if (tu_group->tus == nullptr) - tu_group->tus = new std::vector ; + tu_group->tus = new std::vector; tu_group->tus->push_back (sig_type); prepare_one_comp_unit (cu, type_unit_die, language_minimal); diff --git a/gdb/gdbsupport/btrace-common.c b/gdb/gdbsupport/btrace-common.c index d6d3ab50c9e..608506c34c9 100644 --- a/gdb/gdbsupport/btrace-common.c +++ b/gdb/gdbsupport/btrace-common.c @@ -133,7 +133,7 @@ btrace_data_append (struct btrace_data *dst, case BTRACE_FORMAT_NONE: dst->format = BTRACE_FORMAT_BTS; - dst->variant.bts.blocks = new std::vector ; + dst->variant.bts.blocks = new std::vector; /* Fall-through. */ case BTRACE_FORMAT_BTS: diff --git a/gdb/gdbsupport/btrace-common.h b/gdb/gdbsupport/btrace-common.h index 166d7b18700..09a90485cdc 100644 --- a/gdb/gdbsupport/btrace-common.h +++ b/gdb/gdbsupport/btrace-common.h @@ -141,7 +141,7 @@ struct btrace_data_bts /* Branch trace is represented as a vector of branch trace blocks starting with the most recent block. This needs to be a pointer as we place btrace_data_bts into a union. */ - std::vector *blocks; + std::vector *blocks; }; /* Configuration information to go with the trace data. */ diff --git a/gdb/nat/linux-btrace.c b/gdb/nat/linux-btrace.c index a63973d569d..850fa8df584 100644 --- a/gdb/nat/linux-btrace.c +++ b/gdb/nat/linux-btrace.c @@ -271,11 +271,11 @@ perf_event_sample_ok (const struct perf_event_sample *sample) In case the buffer overflows during sampling, one sample may have its lower part at the end and its upper part at the beginning of the buffer. */ -static std::vector * +static std::vector * perf_event_read_bts (struct btrace_target_info* tinfo, const uint8_t *begin, const uint8_t *end, const uint8_t *start, size_t size) { - std::vector *btrace = new std::vector ; + std::vector *btrace = new std::vector; struct perf_event_sample sample; size_t read = 0; struct btrace_block block = { 0, 0 }; -- 2.30.2