elf/dl-lookup.c: Remove obsolete comment about nested function
authorWill Newton <will.newton@linaro.org>
Wed, 2 Apr 2014 14:23:16 +0000 (15:23 +0100)
committerWill Newton <will.newton@linaro.org>
Thu, 3 Apr 2014 08:58:30 +0000 (09:58 +0100)
The nested function referred to has gone away so remove the
comment. Also move the variable declaration down to where other
variables of a similar lifetime are declared for clarity.

2014-04-03  Will Newton  <will.newton@linaro.org>

* elf/dl-lookup.c (do_lookup_x): Remove comment
referring to nested function and move variable
declarations down to before first use.

ChangeLog
elf/dl-lookup.c

index 7e530ef..e36b689 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2014-04-03  Will Newton  <will.newton@linaro.org>
+
+       * elf/dl-lookup.c (do_lookup_x): Remove comment
+       referring to nested function and move variable
+       declarations down to before first use.
+
 2014-04-02  Joseph Myers  <joseph@codesourcery.com>
 
        [BZ #16799]
index d1b8efc..be6f76f 100644 (file)
@@ -208,11 +208,6 @@ do_lookup_x (const char *undef_name, uint_fast32_t new_hash,
 
   do
     {
-      /* These variables are used in the nested function.  */
-      Elf_Symndx symidx;
-      int num_versions = 0;
-      const ElfW(Sym) *versioned_sym = NULL;
-
       const struct link_map *map = list[i]->l_real;
 
       /* Here come the extra test needed for `_dl_lookup_symbol_skip'.  */
@@ -237,6 +232,10 @@ do_lookup_x (const char *undef_name, uint_fast32_t new_hash,
       if (map->l_nbuckets == 0)
        continue;
 
+      Elf_Symndx symidx;
+      int num_versions = 0;
+      const ElfW(Sym) *versioned_sym = NULL;
+
       /* The tables for this map.  */
       const ElfW(Sym) *symtab = (const void *) D_PTR (map, l_info[DT_SYMTAB]);
       const char *strtab = (const void *) D_PTR (map, l_info[DT_STRTAB]);