From 4fabc3a897d7a6bb9e9594735afa7bf918013bd6 Mon Sep 17 00:00:00 2001 From: Tom Tromey Date: Sat, 3 Jul 2021 11:40:54 -0600 Subject: [PATCH] Use 'bool' more idiomatically in dwarf_decode_lines I noticed a couple of spots related to dwarf_decode_lines where the 'include_p' field was not being used idiomatically -- it is of type bool now, so treat it as such. gdb/ChangeLog 2021-07-03 Tom Tromey * dwarf2/read.c (lnp_state_machine::record_line): Use 'true'. (dwarf_decode_lines): Remove '=='. --- gdb/ChangeLog | 5 +++++ gdb/dwarf2/read.c | 4 ++-- 2 files changed, 7 insertions(+), 2 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 6a5f8b569f8..6a538f9eb86 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,8 @@ +2021-07-03 Tom Tromey + + * dwarf2/read.c (lnp_state_machine::record_line): Use 'true'. + (dwarf_decode_lines): Remove '=='. + 2021-07-02 Tom Tromey * ada-exp.y (chop_selector, chop_separator, write_selectors) diff --git a/gdb/dwarf2/read.c b/gdb/dwarf2/read.c index 760d4319c29..fe82175990e 100644 --- a/gdb/dwarf2/read.c +++ b/gdb/dwarf2/read.c @@ -20997,7 +20997,7 @@ lnp_state_machine::record_line (bool end_sequence) previous version of the code. */ else if (m_op_index == 0 || end_sequence) { - fe->included_p = 1; + fe->included_p = true; if (m_record_lines_p) { /* When we switch files we insert an end maker in the first file, @@ -21368,7 +21368,7 @@ dwarf_decode_lines (struct line_header *lh, const char *comp_dir, /* Now that we're done scanning the Line Header Program, we can create the psymtab of each included file. */ for (auto &file_entry : lh->file_names ()) - if (file_entry.included_p == 1) + if (file_entry.included_p) { gdb::unique_xmalloc_ptr name_holder; const char *include_name = -- 2.30.2