aboutsummaryrefslogtreecommitdiff
path: root/gold/merge.cc
diff options
context:
space:
mode:
authorNick Clifton <nickc@redhat.com>2009-12-11 13:42:17 +0000
committerNick Clifton <nickc@redhat.com>2009-12-11 13:42:17 +0000
commit91d6fa6a035cc7d0b7be5c99c194a64cb80924b0 (patch)
tree214507c313b77d619b52afcae2af0b02c9fa700b /gold/merge.cc
parent01fe1b4183324882e88e8c64748bffdc69ea3a9c (diff)
downloadgdb-91d6fa6a035cc7d0b7be5c99c194a64cb80924b0.zip
gdb-91d6fa6a035cc7d0b7be5c99c194a64cb80924b0.tar.gz
gdb-91d6fa6a035cc7d0b7be5c99c194a64cb80924b0.tar.bz2
Add -Wshadow to the gcc command line options used when compiling the binutils.
Fix up all warnings generated by the addition of this switch.
Diffstat (limited to 'gold/merge.cc')
-rw-r--r--gold/merge.cc32
1 files changed, 16 insertions, 16 deletions
diff --git a/gold/merge.cc b/gold/merge.cc
index 3d96921..654f349 100644
--- a/gold/merge.cc
+++ b/gold/merge.cc
@@ -289,10 +289,10 @@ Merge_map::is_merge_section_for(const Relobj* object, unsigned int shndx) const
bool
Output_merge_base::do_output_offset(const Relobj* object,
unsigned int shndx,
- section_offset_type offset,
+ section_offset_type off,
section_offset_type* poutput) const
{
- return this->merge_map_.get_output_offset(object, shndx, offset, poutput);
+ return this->merge_map_.get_output_offset(object, shndx, off, poutput);
}
// Return whether this is the merge section for SHNDX in OBJECT.
@@ -354,10 +354,10 @@ Output_merge_data::Merge_data_eq::operator()(Merge_data_key k1,
void
Output_merge_data::add_constant(const unsigned char* p)
{
- section_size_type entsize = convert_to_section_size_type(this->entsize());
- section_size_type addralign =
+ section_size_type ent_size = convert_to_section_size_type(this->entsize());
+ section_size_type addr_align =
convert_to_section_size_type(this->addralign());
- section_size_type addsize = std::max(entsize, addralign);
+ section_size_type addsize = std::max(ent_size, addr_align);
if (this->len_ + addsize > this->alc_)
{
if (this->alc_ == 0)
@@ -369,9 +369,9 @@ Output_merge_data::add_constant(const unsigned char* p)
gold_nomem();
}
- memcpy(this->p_ + this->len_, p, entsize);
- if (addsize > entsize)
- memset(this->p_ + this->len_ + entsize, 0, addsize - entsize);
+ memcpy(this->p_ + this->len_, p, ent_size);
+ if (addsize > ent_size)
+ memset(this->p_ + this->len_ + ent_size, 0, addsize - ent_size);
this->len_ += addsize;
}
@@ -386,14 +386,14 @@ Output_merge_data::do_add_input_section(Relobj* object, unsigned int shndx)
section_size_type len;
const unsigned char* p = object->section_contents(shndx, &len, false);
- section_size_type entsize = convert_to_section_size_type(this->entsize());
+ section_size_type ent_size = convert_to_section_size_type(this->entsize());
- if (len % entsize != 0)
+ if (len % ent_size != 0)
return false;
- this->input_count_ += len / entsize;
+ this->input_count_ += len / ent_size;
- for (section_size_type i = 0; i < len; i += entsize, p += entsize)
+ for (section_size_type i = 0; i < len; i += ent_size, p += ent_size)
{
// Add the constant to the section contents. If we find that it
// is already in the hash table, we will remove it again.
@@ -406,12 +406,12 @@ Output_merge_data::do_add_input_section(Relobj* object, unsigned int shndx)
if (!ins.second)
{
// Key was already present. Remove the copy we just added.
- this->len_ -= entsize;
+ this->len_ -= ent_size;
k = *ins.first;
}
// Record the offset of this constant in the output section.
- this->add_mapping(object, shndx, i, entsize, k);
+ this->add_mapping(object, shndx, i, ent_size, k);
}
return true;
@@ -531,9 +531,9 @@ Output_merge_string<Char_type>::finalize_merged_data()
p != this->merged_strings_.end();
++p)
{
- section_offset_type offset =
+ section_offset_type soffset =
this->stringpool_.get_offset_from_key(p->stringpool_key);
- this->add_mapping(p->object, p->shndx, p->offset, p->length, offset);
+ this->add_mapping(p->object, p->shndx, p->offset, p->length, soffset);
}
// Save some memory. This also ensures that this function will work