diff options
author | Jan Beulich <jbeulich@suse.com> | 2025-03-07 11:26:43 +0100 |
---|---|---|
committer | Jan Beulich <jbeulich@suse.com> | 2025-03-07 11:26:43 +0100 |
commit | e83aca0906f40c9dc0cf06dd0b743a5ad4df1ceb (patch) | |
tree | ce4acb9f5c02cce010481c61dcc0d48bf437dc73 | |
parent | cdd8492b05979842a1660e99862cd1680e52b28d (diff) | |
download | binutils-e83aca0906f40c9dc0cf06dd0b743a5ad4df1ceb.zip binutils-e83aca0906f40c9dc0cf06dd0b743a5ad4df1ceb.tar.gz binutils-e83aca0906f40c9dc0cf06dd0b743a5ad4df1ceb.tar.bz2 |
gas: centralize declaration of listing_tail
Besides it being somewhat off to have three decls scattered across the
code base, it is generally bad practice for the definition of a symbol
to not also observe its declaration (making sure the two won't go out of
sync).
-rw-r--r-- | gas/config/tc-arm.c | 1 | ||||
-rw-r--r-- | gas/frags.c | 5 | ||||
-rw-r--r-- | gas/listing.h | 2 | ||||
-rw-r--r-- | gas/read.c | 1 |
4 files changed, 3 insertions, 6 deletions
diff --git a/gas/config/tc-arm.c b/gas/config/tc-arm.c index ed910b8..822fd3d 100644 --- a/gas/config/tc-arm.c +++ b/gas/config/tc-arm.c @@ -3260,7 +3260,6 @@ s_thumb_set (int equiv) for this symbol. */ if (listing & LISTING_SYMBOLS) { - extern struct list_info_struct * listing_tail; fragS * dummy_frag = (fragS * ) xmalloc (sizeof (fragS)); memset (dummy_frag, 0, sizeof (fragS)); diff --git a/gas/frags.c b/gas/frags.c index d067bb4..6684e27 100644 --- a/gas/frags.c +++ b/gas/frags.c @@ -192,10 +192,7 @@ frag_new (size_t old_frags_var_max_size frchP->frch_last = frag_now; #ifndef NO_LISTING - { - extern struct list_info_struct *listing_tail; - frag_now->line = listing_tail; - } + frag_now->line = listing_tail; #endif gas_assert (frchain_now->frch_last == frag_now); diff --git a/gas/listing.h b/gas/listing.h index 791d584..cd49498 100644 --- a/gas/listing.h +++ b/gas/listing.h @@ -62,6 +62,8 @@ extern int listing_lhs_width_second; extern int listing_lhs_cont_lines; extern int listing_rhs_width; +extern struct list_info_struct *listing_tail; + #endif /* __listing_h__ */ /* end of listing.h */ @@ -3270,7 +3270,6 @@ assign_symbol (char *name, int mode) for this symbol. */ if (listing & LISTING_SYMBOLS) { - extern struct list_info_struct *listing_tail; fragS *dummy_frag = notes_calloc (1, sizeof (*dummy_frag)); dummy_frag->line = listing_tail; dummy_frag->fr_symbol = symbolP; |