diff mbox

Update comments in scan_partial_symbols and add_partial_subprogram

Message ID 87egw4hite.fsf@codesourcery.com
State New
Headers show

Commit Message

Yao Qi Aug. 25, 2014, 12:09 p.m. UTC
Pedro Alves <palves@redhat.com> writes:

> Hmm.  I still can't parse this though.  It's actually the "to the" part
> that seems to miss a verb to me.  Isn't a "Set" missing instead of
> "are"?  Like:
>
>   that this subprogram contains.  If SET_ADDRMAP is true, record the
>   covered ranges in the addrmap.  Set *LOWPC and *HIGHPC to the lowest and
>   highest PC values found in PDI.

Sigh, my brain stopped working!  Sorry about that.  Fix it as you suggested.
diff mbox

Patch

diff --git a/gdb/dwarf2read.c b/gdb/dwarf2read.c
index df6deef..4720198 100644
--- a/gdb/dwarf2read.c
+++ b/gdb/dwarf2read.c
@@ -7007,8 +7007,8 @@  add_partial_module (struct partial_die_info *pdi, CORE_ADDR *lowpc,
    symbol for that subprogram.  When the CU language allows it, this
    routine also defines a partial symbol for each nested subprogram
    that this subprogram contains.  If SET_ADDRMAP is true, record the
-   covered ranges in the addrmap.  *LOWPC and *HIGHPC to the lowest and
-   highest PC values are found in PDI.
+   covered ranges in the addrmap.  Set *LOWPC and *HIGHPC to the lowest
+   and highest PC values found in PDI.
 
    PDI may also be a lexical block, in which case we simply search
    recursively for subprograms defined inside that lexical block.