aboutsummaryrefslogtreecommitdiff
path: root/gold/resolve.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/resolve.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/resolve.cc')
-rw-r--r--gold/resolve.cc38
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);