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/resolve.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/resolve.cc')
-rw-r--r-- | gold/resolve.cc | 38 |
1 files changed, 19 insertions, 19 deletions
diff --git a/gold/resolve.cc b/gold/resolve.cc index 89b10b9..1d77a92 100644 --- a/gold/resolve.cc +++ b/gold/resolve.cc @@ -37,9 +37,9 @@ namespace gold // VERSION. Update the VERSION_ field accordingly. inline void -Symbol::override_version(const char* version) +Symbol::override_version(const char* aversion) { - if (version == NULL) + if (aversion == NULL) { // This is the case where this symbol is NAME/VERSION, and the // version was not marked as hidden. That makes it the default @@ -49,7 +49,7 @@ Symbol::override_version(const char* version) // override NAME/VERSION as well. They are already the same // Symbol structure. Setting the VERSION_ field to NULL ensures // that it will be output with the correct, empty, version. - this->version_ = version; + this->version_ = aversion; } else { @@ -58,8 +58,8 @@ Symbol::override_version(const char* version) // overriding NAME. If VERSION_ONE and VERSION_TWO are // different, then this can only happen when VERSION_ONE is NULL // and VERSION_TWO is not hidden. - gold_assert(this->version_ == version || this->version_ == NULL); - this->version_ = version; + gold_assert(this->version_ == aversion || this->version_ == NULL); + this->version_ = aversion; } } @@ -67,19 +67,19 @@ Symbol::override_version(const char* version) // is VISIBILITY. Updated the VISIBILITY_ field accordingly. inline void -Symbol::override_visibility(elfcpp::STV visibility) +Symbol::override_visibility(elfcpp::STV avisibility) { // The rule for combining visibility is that we always choose the // most constrained visibility. In order of increasing constraint, // visibility goes PROTECTED, HIDDEN, INTERNAL. This is the reverse // of the numeric values, so the effect is that we always want the // smallest non-zero value. - if (visibility != elfcpp::STV_DEFAULT) + if (avisibility != elfcpp::STV_DEFAULT) { if (this->visibility_ == elfcpp::STV_DEFAULT) - this->visibility_ = visibility; - else if (this->visibility_ > visibility) - this->visibility_ = visibility; + this->visibility_ = avisibility; + else if (this->visibility_ > avisibility) + this->visibility_ = avisibility; } } @@ -89,18 +89,18 @@ template<int size, bool big_endian> void Symbol::override_base(const elfcpp::Sym<size, big_endian>& sym, unsigned int st_shndx, bool is_ordinary, - Object* object, const char* version) + Object* aobject, const char* aversion) { gold_assert(this->source_ == FROM_OBJECT); - this->u_.from_object.object = object; - this->override_version(version); + this->u_.from_object.object = aobject; + this->override_version(aversion); this->u_.from_object.shndx = st_shndx; this->is_ordinary_shndx_ = is_ordinary; this->type_ = sym.get_st_type(); this->binding_ = sym.get_st_bind(); this->override_visibility(sym.get_st_visibility()); this->nonvis_ = sym.get_st_nonvis(); - if (object->is_dynamic()) + if (aobject->is_dynamic()) this->in_dyn_ = true; else this->in_reg_ = true; @@ -113,9 +113,9 @@ template<bool big_endian> void Sized_symbol<size>::override(const elfcpp::Sym<size, big_endian>& sym, unsigned st_shndx, bool is_ordinary, - Object* object, const char* version) + Object* aobject, const char* aversion) { - this->override_base(sym, st_shndx, is_ordinary, object, version); + this->override_base(sym, st_shndx, is_ordinary, aobject, aversion); this->value_ = sym.get_st_value(); this->symsize_ = sym.get_st_size(); } @@ -128,9 +128,9 @@ void Symbol_table::override(Sized_symbol<size>* tosym, const elfcpp::Sym<size, big_endian>& fromsym, unsigned int st_shndx, bool is_ordinary, - Object* object, const char* version) + Object* aobject, const char* aversion) { - tosym->override(fromsym, st_shndx, is_ordinary, object, version); + tosym->override(fromsym, st_shndx, is_ordinary, aobject, aversion); if (tosym->has_alias()) { Symbol* sym = this->weak_aliases_[tosym]; @@ -138,7 +138,7 @@ Symbol_table::override(Sized_symbol<size>* tosym, Sized_symbol<size>* ssym = this->get_sized_symbol<size>(sym); do { - ssym->override(fromsym, st_shndx, is_ordinary, object, version); + ssym->override(fromsym, st_shndx, is_ordinary, aobject, aversion); sym = this->weak_aliases_[ssym]; gold_assert(sym != NULL); ssym = this->get_sized_symbol<size>(sym); |