libs/libc/elf: Fix the nxstyle errors around the FDPIC changes.

The FDPIC work touches these files, and nxstyle reports errors on the lines
around every hunk, which fails the check job.  The errors are older than
this series: a switch body indented two columns too deep in elf_symbols.c,
and declarations with no blank line after them.

Whitespace and one reworded comment, no change in behaviour.

Signed-off-by: Marco Casaroli <marco.casaroli@gmail.com>

Author: casaroli

libs/libc/elf/elf_insert.c

@@ -85,6 +85,7 @@ void libelf_dumploadinfo(FAR struct mod_loadinfo_s *loadinfo)
       for (i = 0; i < loadinfo->ehdr.e_shnum; i++)
         {
           FAR Elf_Shdr *shdr = &loadinfo->shdr[i];
+
           binfo("Sections %d:\n", i);
 #  ifdef CONFIG_ARCH_USE_SEPARATED_SECTION
           if (loadinfo->ehdr.e_type == ET_REL)
@@ -420,27 +421,27 @@ FAR void *libelf_insert(FAR const char *filename, FAR const char *modname)
       case ET_REL :
       case ET_DYN :
 
-          /* Process any preinit_array entries */
+        /* Process any preinit_array entries */
 
-          array = (FAR void (**)(void))loadinfo.preiarr;
-          for (i = 0; i < loadinfo.nprei; i++)
-            {
-              array[i]();
-            }
+        array = (FAR void (**)(void))loadinfo.preiarr;
+        for (i = 0; i < loadinfo.nprei; i++)
+          {
+            array[i]();
+          }
 
-          /* Process any init_array entries */
+        /* Process any init_array entries */
 
-          array = (FAR void (**)(void))loadinfo.initarr;
-          for (i = 0; i < loadinfo.ninit; i++)
-            {
-              array[i]();
-            }
+        array = (FAR void (**)(void))loadinfo.initarr;
+        for (i = 0; i < loadinfo.ninit; i++)
+          {
+            array[i]();
+          }
 
-          modp->initarr = loadinfo.initarr;
-          modp->ninit = loadinfo.ninit;
-          modp->finiarr = loadinfo.finiarr;
-          modp->nfini = loadinfo.nfini;
-          break;
+        modp->initarr = loadinfo.initarr;
+        modp->ninit = loadinfo.ninit;
+        modp->finiarr = loadinfo.finiarr;
+        modp->nfini = loadinfo.nfini;
+        break;
     }
 
   /* Add the new module entry to the registry */

libs/libc/elf/elf_load.c

@@ -408,6 +408,7 @@ static inline int libelf_loadfile(FAR struct mod_loadinfo_s *loadinfo)
               else
                 {
                   size_t bsssize = phdr->p_memsz - phdr->p_filesz;
+
                   ret = libelf_read(loadinfo, data, phdr->p_filesz,
                                     phdr->p_offset);
                   memset(data + phdr->p_filesz, 0, bsssize);

libs/libc/elf/elf_symbols.c

@@ -220,6 +220,7 @@ static int libelf_symcallback(FAR struct module_s *modp, FAR void *arg)
 
 #if CONFIG_LIBC_ELF_MAXDEPEND > 0
       int ret = libelf_depend(exportinfo->modp, modp);
+
       if (ret < 0)
         {
           berr("ERROR: libelf_depend failed: %d\n", ret);
@@ -354,108 +355,108 @@ int libelf_symvalue(FAR struct module_s *modp,
 
   switch (sym->st_shndx)
     {
-    case SHN_COMMON:
-      {
-        /* NuttX ELF modules should be compiled with -fno-common. */
-
-        berr("ERROR: SHN_COMMON: Re-compile with -fno-common\n");
-        return -ENOSYS;
-      }
-
-    case SHN_ABS:
-      {
-        /* st_value already holds the correct value */
-
-        binfo("SHN_ABS: st_value=%08lx\n", (long)sym->st_value);
-        return OK;
-      }
-
-    case SHN_UNDEF:
-      {
-        /* Get the name of the undefined symbol */
-
-        ret = libelf_symname(loadinfo, sym, sh_offset);
-        if (ret < 0)
-          {
-            /* There are a few relocations for a few architectures that do
-             * no depend upon a named symbol.  We don't know if that is the
-             * case here, but return and special error to the caller to
-             * indicate the nameless symbol.
-             */
-
-            berr("ERROR: SHN_UNDEF: Failed to get symbol name: %d\n", ret);
-            return ret;
-          }
-
-        /* First check if the symbol is exported by an installed module.
-         * Newest modules are installed at the head of the list.  Therefore,
-         * if the symbol is exported by numerous modules, then the most
-         * recently installed will take precedence.
-         */
-
-        exportinfo.name   = (FAR const char *)loadinfo->iobuffer;
-        exportinfo.modp   = modp;
-        exportinfo.symbol = NULL;
-
-        ret = libelf_registry_foreach(libelf_symcallback,
-                                      (FAR void *)&exportinfo);
-        if (ret < 0)
-          {
-            berr("ERROR: libelf_symcallback failed: %d\n", ret);
-            return ret;
-          }
-
-        symbol = exportinfo.symbol;
-
-        /* If the symbol is not exported by any module, then check if the
-         * base code exports a symbol of this name.
-         */
-
-        if (symbol == NULL)
-          {
-            symbol = symtab_findbyname(exports, exportinfo.name,
-                                       nexports);
-          }
-
-        /* Was the symbol found from any exporter? */
-
-        if (symbol == NULL)
-          {
-            berr("ERROR: SHN_UNDEF: Exported symbol \"%s\" not found\n",
-                 loadinfo->iobuffer);
-            return -ENOENT;
-          }
-
-        /* Yes... add the exported symbol value to the ELF symbol tablei
-         * entry
-         */
-
-        binfo("SHN_UNDEF: name=%s "
-              "%08" PRIxPTR "+%08" PRIxPTR "=%08" PRIxPTR "\n",
-              loadinfo->iobuffer,
-              (uintptr_t)sym->st_value, (uintptr_t)symbol->sym_value,
-              (uintptr_t)(sym->st_value + (uintptr_t)symbol->sym_value));
-
-        sym->st_value += ((uintptr_t)symbol->sym_value);
-      }
-      break;
-
-    default:
-      {
-        secbase = loadinfo->shdr[sym->st_shndx].sh_addr;
-
-        binfo("Other[%d]: %08" PRIxPTR "+%08" PRIxPTR "=%08" PRIxPTR "\n",
-              sym->st_shndx,
-              (uintptr_t)sym->st_value, secbase,
-              (uintptr_t)(sym->st_value + secbase));
-
-        sym->st_value += secbase;
-        if (loadinfo->gotindex >= 0)
-          {
-            sym->st_value -= loadinfo->shdr[sym->st_shndx].sh_offset;
-          }
-      }
-      break;
+      case SHN_COMMON:
+        {
+          /* NuttX ELF modules should be compiled with -fno-common. */
+
+          berr("ERROR: SHN_COMMON: Re-compile with -fno-common\n");
+          return -ENOSYS;
+        }
+
+      case SHN_ABS:
+        {
+          /* st_value already holds the correct value */
+
+          binfo("SHN_ABS: st_value=%08lx\n", (long)sym->st_value);
+          return OK;
+        }
+
+      case SHN_UNDEF:
+        {
+          /* Get the name of the undefined symbol */
+
+          ret = libelf_symname(loadinfo, sym, sh_offset);
+          if (ret < 0)
+            {
+              /* There are a few relocations for a few architectures that do
+               * no depend upon a named symbol.  We don't know if that is the
+               * case here, but return and special error to the caller to
+               * indicate the nameless symbol.
+               */
+
+              berr("ERROR: SHN_UNDEF: Failed to get symbol name: %d\n", ret);
+              return ret;
+            }
+
+          /* First check if the symbol is exported by an installed module.
+           * Newest modules are installed at the head of the list.  So if
+           * the symbol is exported by numerous modules, then the most
+           * recently installed will take precedence.
+           */
+
+          exportinfo.name   = (FAR const char *)loadinfo->iobuffer;
+          exportinfo.modp   = modp;
+          exportinfo.symbol = NULL;
+
+          ret = libelf_registry_foreach(libelf_symcallback,
+                                        (FAR void *)&exportinfo);
+          if (ret < 0)
+            {
+              berr("ERROR: libelf_symcallback failed: %d\n", ret);
+              return ret;
+            }
+
+          symbol = exportinfo.symbol;
+
+          /* If the symbol is not exported by any module, then check if the
+           * base code exports a symbol of this name.
+           */
+
+          if (symbol == NULL)
+            {
+              symbol = symtab_findbyname(exports, exportinfo.name,
+                                         nexports);
+            }
+
+          /* Was the symbol found from any exporter? */
+
+          if (symbol == NULL)
+            {
+              berr("ERROR: SHN_UNDEF: Exported symbol \"%s\" not found\n",
+                   loadinfo->iobuffer);
+              return -ENOENT;
+            }
+
+          /* Yes... add the exported symbol value to the ELF symbol tablei
+           * entry
+           */
+
+          binfo("SHN_UNDEF: name=%s "
+                "%08" PRIxPTR "+%08" PRIxPTR "=%08" PRIxPTR "\n",
+                loadinfo->iobuffer,
+                (uintptr_t)sym->st_value, (uintptr_t)symbol->sym_value,
+                (uintptr_t)(sym->st_value + (uintptr_t)symbol->sym_value));
+
+          sym->st_value += ((uintptr_t)symbol->sym_value);
+        }
+        break;
+
+      default:
+        {
+          secbase = loadinfo->shdr[sym->st_shndx].sh_addr;
+
+          binfo("Other[%d]: %08" PRIxPTR "+%08" PRIxPTR "=%08" PRIxPTR "\n",
+                sym->st_shndx,
+                (uintptr_t)sym->st_value, secbase,
+                (uintptr_t)(sym->st_value + secbase));
+
+          sym->st_value += secbase;
+          if (loadinfo->gotindex >= 0)
+            {
+              sym->st_value -= loadinfo->shdr[sym->st_shndx].sh_offset;
+            }
+        }
+        break;
     }
 
   return OK;
@@ -596,6 +597,7 @@ static int findep(FAR const void *c1, FAR const void *c2)
 {
   FAR const struct eptable_s *m1 = (FAR const struct eptable_s *)c1;
   FAR const struct eptable_s *m2 = (FAR const struct eptable_s *)c2;
+
   return strcmp((FAR const char *)m1->epname, (FAR const char *)m2->epname);
 }