diff options
author | Nick Clifton <nickc@redhat.com> | 2009-12-11 13:42:17 +0000 |
---|---|---|
committer | Nick Clifton <nickc@redhat.com> | 2009-12-11 13:42:17 +0000 |
commit | 91d6fa6a035cc7d0b7be5c99c194a64cb80924b0 (patch) | |
tree | 214507c313b77d619b52afcae2af0b02c9fa700b /gold/merge.cc | |
parent | 01fe1b4183324882e88e8c64748bffdc69ea3a9c (diff) | |
download | gdb-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.cc | 32 |
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 |