]> git.ipfire.org Git - thirdparty/coreutils.git/commitdiff
ls: omit cast from gobble_file
authorPaul Eggert <eggert@cs.ucla.edu>
Wed, 2 Oct 2024 18:22:03 +0000 (11:22 -0700)
committerPaul Eggert <eggert@cs.ucla.edu>
Thu, 3 Oct 2024 06:03:04 +0000 (23:03 -0700)
* src/ls.c (gobble_file): Minor refactoring.  Last arg is now null
pointer, not "", for no directory.  All callers changed.
Avoid need for cast from char const * to char *.

src/ls.c

index 12af327947e24b6a24ec2d8e16f17ff1fabd0781..57f1621ff349066c73b10ab2e052e5e8d7cfacac 100644 (file)
--- a/src/ls.c
+++ b/src/ls.c
@@ -1770,13 +1770,13 @@ main (int argc, char **argv)
   if (n_files <= 0)
     {
       if (immediate_dirs)
-        gobble_file (".", directory, NOT_AN_INODE_NUMBER, true, "");
+        gobble_file (".", directory, NOT_AN_INODE_NUMBER, true, nullptr);
       else
         queue_directory (".", nullptr, true);
     }
   else
     do
-      gobble_file (argv[i++], unknown, NOT_AN_INODE_NUMBER, true, "");
+      gobble_file (argv[i++], unknown, NOT_AN_INODE_NUMBER, true, nullptr);
     while (i < argc);
 
   if (cwd_n_used)
@@ -3405,16 +3405,17 @@ gobble_file (char const *name, enum filetype type, ino_t inode,
 
     {
       /* Absolute name of this file.  */
-      char *full_name;
+      char const *full_name;
       bool do_deref;
       int err;
 
-      if (name[0] == '/' || dirname[0] == 0)
-        full_name = (char *) name;
+      if (name[0] == '/' || !dirname)
+        full_name = name;
       else
         {
-          full_name = alloca (strlen (name) + strlen (dirname) + 2);
-          attach (full_name, dirname, name);
+          char *p = alloca (strlen (name) + strlen (dirname) + 2);
+          attach (p, dirname, name);
+          full_name = p;
         }
 
       if (print_hyperlink)