toggle executable and show file mode string and change wpprintw to valist arg

ccc.c

@@ -23,11 +23,13 @@ int mkdir_p(const char *destdir);
 void populate_files(const char *path, int ftype);
 int get_directory_size(const char *fpath, const struct stat *sb, int typeflag, struct FTW *ftwbuf);
 void add_file_stat(char *filepath, int ftype);
+char *get_file_mode(mode_t mode);
 void highlight_current_line();
 void show_file_content();
 void edit_file();
+void toggle_executable();
 int write_last_d();
-void wpprintw(const char *line);
+void wpprintw(const char *fmt, ...);
 void init_windows();
 void draw_border_title(WINDOW *window, bool active);
 
@@ -105,8 +107,8 @@ int main(int argc, char** argv)
         getcwd(cwd, PATH_MAX);
     }
     p_cwd = memalloc(PATH_MAX * sizeof(char));
-    populate_files(cwd, 0);
     start_ccc();
+    populate_files(cwd, 0);
 
     int ch, ch2;
     while (1) {
@@ -281,6 +283,10 @@ int main(int argc, char** argv)
                 change_dir(cwd, 0, 0);
                 break;
 
+            case 'X':
+                toggle_executable();
+                break;
+
             /* mark one file */
             case SPACE:
                 add_file_stat(files->items[current_selection].path, 1);
@@ -340,6 +346,7 @@ int main(int argc, char** argv)
                 delwin(panel);
                 endwin();
                 start_ccc(); 
+                highlight_current_line();
                 break;
             default:
                 break;
@@ -365,7 +372,6 @@ void start_ccc()
 {
     half_width = COLS / 2;
     init_windows();
-    highlight_current_line();
 }
 
 /*
@@ -385,9 +391,8 @@ void change_dir(const char *buf, int selection, int ftype)
     strcpy(cwd, buf_dup);
     arraylist_free(files);
     files = arraylist_init(100);
-    populate_files(cwd, ftype);
     current_selection = selection;
-    highlight_current_line();
+    populate_files(cwd, ftype);
 }
 
 /*
@@ -473,6 +478,7 @@ void populate_files(const char *path, int ftype)
         }
         closedir(dp);
         wrefresh(directory_content);
+        highlight_current_line();
     } else {
         perror("ccc");
     }
@@ -564,8 +570,11 @@ void add_file_stat(char *filepath, int ftype)
     /* display sizes */
     sprintf(size, "%.3g%s", bytes, units[unit]);
 
-    char *total_stat = memalloc(45 * sizeof(char));
+    /* get file mode string */
-    snprintf(total_stat, 45, "%-18s %-8s", time, size);
+    char *mode_str = get_file_mode(file_stat.st_mode);
+
+    char *total_stat = memalloc(56 * sizeof(char));
+    snprintf(total_stat, 56, "%s %s %-8s", mode_str, time, size);
     total_stat[strlen(total_stat)] = '\0';
 
     arraylist_add(files, filepath, total_stat, type, color, false, false);
@@ -574,8 +583,29 @@ void add_file_stat(char *filepath, int ftype)
     free(size);
     free(total_stat);
     free(type);
+    free(mode_str);
 }
 
+/*
+ * get file mode string from stat mode
+ * eg: drwxr-sr-x
+ */
+char *get_file_mode(mode_t mode)
+{
+    char *mode_str = memalloc(sizeof(char) * 11);
+    mode_str[0] = S_ISDIR(mode) ? 'd' : '-'; // Check if it's a directory
+    mode_str[1] = (mode & S_IRUSR) ? 'r' : '-';
+    mode_str[2] = (mode & S_IWUSR) ? 'w' : '-';
+    mode_str[3] = (mode & S_IXUSR) ? 'x' : '-';
+    mode_str[4] = (mode & S_IRGRP) ? 'r' : '-';
+    mode_str[5] = (mode & S_IWGRP) ? 'w' : '-';
+    mode_str[6] = (mode & S_IXGRP) ? 'x' : '-';
+    mode_str[7] = (mode & S_IROTH) ? 'r' : '-';
+    mode_str[8] = (mode & S_IWOTH) ? 'w' : '-';
+    mode_str[9] = (mode & S_IXOTH) ? 'x' : '-';
+    mode_str[10] = '\0'; // Null terminator
+    return mode_str;
+}
 
 /*
  * Highlight current line by reversing the color
@@ -595,7 +625,7 @@ void highlight_current_line()
     /* calculate range of files to show */
     long range = files->length;
     /* not highlight if no files in directory */
-    if (range == 0) {
+    if (range == 0 && errno == 0) {
         #if DRAW_PREVIEW
             wprintw(preview_content, "empty directory");
             wrefresh(preview_content);
@@ -616,9 +646,6 @@ void highlight_current_line()
         if ((overflow == 0 && i == current_selection) || (overflow != 0 && i == current_selection)) {
             wattron(directory_content, A_REVERSE);
 
-            /* update the panel */
-            wclear(panel);
-            
             /* check for marked files */
             long num_marked = marked->length;
             if (num_marked > 0) {
@@ -627,9 +654,9 @@ void highlight_current_line()
                 char *selected = memalloc((m_len + 1) * sizeof(char));
 
                 snprintf(selected, m_len + 1, "[%ld] selected", num_marked);
-                wprintw(panel, "(%ld/%ld) %s %s", current_selection + 1, files->length, selected, cwd);
+                wpprintw("(%ld/%ld) %s %s", current_selection + 1, files->length, selected, cwd);
             } else  {
-                wprintw(panel, "(%ld/%ld) %s", current_selection + 1, files->length, cwd);
+                wpprintw("(%ld/%ld) %s", current_selection + 1, files->length, cwd);
             }
         }
         /* print the actual filename and stats */
@@ -668,6 +695,9 @@ void highlight_current_line()
         show_file_content();
     #endif
     wrefresh(preview_content);
+    #if DRAW_BORDERS
+        draw_border_title(preview_border, true);
+    #endif
 }
 
 /*
@@ -686,10 +716,7 @@ void show_file_content()
         mvwprintw(preview_content, 0, 0, "Unable to read %s", current_file.path);
         return;
     }
-    #if DRAW_BORDERS
+    
-        draw_border_title(preview_border, true);
-    #endif
-
     int c;
     /* check if its binary */
     while ((c = fgetc(file)) != EOF) {
@@ -745,6 +772,22 @@ void edit_file()
     }
 }
 
+void toggle_executable()
+{
+    file current_file = files->items[current_selection];
+    struct stat st;
+    if (stat(current_file.path, &st) == -1) {
+        wpprintw("stat failed: %s\n", strerror(errno));
+    }
+    if (strncmp(current_file.type, "DIR", 3) == 0)
+        return;
+    /* chmod by xor executable bits */
+    if (chmod(current_file.path, st.st_mode ^ (S_IXUSR | S_IXGRP | S_IXOTH)) == -1) {
+        wpprintw("Error toggling executable: %s", strerror(errno));
+    }
+
+}
+
 int write_last_d()
 {
     #ifdef LAST_D
@@ -785,10 +828,13 @@ int write_last_d()
 /*
  * Print line to the panel
  */
-void wpprintw(const char *line)
+void wpprintw(const char *fmt, ...)
 {
+    va_list args;
+    va_start(args, fmt);
     wclear(panel);
-    wprintw(panel, "%s", line);
+    vw_printw(panel, fmt, args);
+    va_end(args);
     wrefresh(panel);
 }