]> Sergey Matveev's repositories - nnn.git/commitdiff
Further gitstatus cleanup/fixes (#1046)
authorluukvbaal <31730729+luukvbaal@users.noreply.github.com>
Thu, 3 Jun 2021 09:01:27 +0000 (11:01 +0200)
committerGitHub <noreply@github.com>
Thu, 3 Jun 2021 09:01:27 +0000 (11:01 +0200)
patches/gitstatus/mainline.diff
patches/gitstatus/namefirst.diff
patches/namefirst/mainline.diff

index 1179e8c66951bfb6cecc0dea9f2f43887823b349..a857adacc15a317d3e89d3d6675cb4986f6709c7 100644 (file)
@@ -3,26 +3,25 @@
 # Dependencies: libgit2
 #
 # Authors: @crides, Luuk van Baal
+
 diff --git a/src/nnn.c b/src/nnn.c
-index 562622f..96d946a 100644
+index d83d7f6..b23218e 100644
 --- a/src/nnn.c
 +++ b/src/nnn.c
-@@ -127,6 +127,8 @@
- #include "qsort.h"
- #endif
-
+@@ -105,6 +105,7 @@
+ #include <wchar.h>
+ #include <pwd.h>
+ #include <grp.h>
 +#include <git2.h>
-+
- /* Macro definitions */
- #define VERSION "4.0"
- #define GENERAL_INFO "BSD 2-Clause\nhttps://github.com/jarun/nnn"
-@@ -245,6 +247,18 @@ typedef unsigned char uchar_t;
+
+ #if !defined(alloca) && defined(__GNUC__)
+ /*
+@@ -244,6 +245,16 @@ typedef unsigned int uint_t;
+ typedef unsigned char uchar_t;
  typedef unsigned short ushort_t;
  typedef unsigned long long ulong_t;
-
 +typedef enum {
-+      GIT_COLUMN_STATUS_NONE = 0,
-+      GIT_COLUMN_STATUS_UNMOD,
++      GIT_COLUMN_STATUS_UNMOD = 0,
 +      GIT_COLUMN_STATUS_NEW,
 +      GIT_COLUMN_STATUS_MODIFIED,
 +      GIT_COLUMN_STATUS_DELETED,
@@ -31,11 +30,10 @@ index 562622f..96d946a 100644
 +      GIT_COLUMN_STATUS_IGNORED,
 +      GIT_COLUMN_STATUS_CONFLICTED,
 +} git_column_status_t;
-+
+
  /* STRUCTURES */
 
- /* Directory entry */
-@@ -258,6 +272,8 @@ typedef struct entry {
+@@ -258,6 +269,8 @@ typedef struct entry {
                ulong_t blocks : 40; /* 5 bytes (enough for 512 TiB in 512B blocks allocated) */
                ulong_t nlen : 16; /* 2 bytes (length of file name) */
                ulong_t flags : 8; /* 1 byte (flags specific to the file) */
@@ -44,7 +42,7 @@ index 562622f..96d946a 100644
        };
  #ifndef NOUG
        uid_t uid; /* 4 bytes */
-@@ -362,7 +378,18 @@ typedef struct {
+@@ -362,7 +375,18 @@ typedef struct {
  } session_header_t;
  #endif
 
@@ -63,9 +61,9 @@ index 562622f..96d946a 100644
 
  /* Configuration, contexts */
  static settings cfg = {
-@@ -809,6 +836,92 @@ static void notify_fifo(bool force);
-
- /* Functions */
+@@ -3464,6 +3488,79 @@ static char *get_kv_val(kv *kvarr, char *buf, int key, uchar_t max, uchar_t id)
+       return NULL;
+ }
 
 +static git_column_status_t git_get_indexed_status(const uint32_t status) {
 +      if (status & GIT_STATUS_INDEX_NEW)         return GIT_COLUMN_STATUS_NEW;
@@ -89,7 +87,6 @@ index 562622f..96d946a 100644
 +
 +static void print_gitstatus(git_column_status_t status) {
 +      switch (status) {
-+              case GIT_COLUMN_STATUS_NONE:         break;
 +              case GIT_COLUMN_STATUS_UNMOD:        addch('-' | 0); break;
 +              case GIT_COLUMN_STATUS_NEW:          addch('N' | (g_state.oldcolor ? COLOR_PAIR(1) : COLOR_PAIR(C_EXE))); break;
 +              case GIT_COLUMN_STATUS_MODIFIED:     addch('M' | (g_state.oldcolor ? COLOR_PAIR(1) : COLOR_PAIR(4))); break;
@@ -101,42 +98,30 @@ index 562622f..96d946a 100644
 +      }
 +}
 +
-+static bool starts_with(const char *const s, const char *const start) {
-+      const size_t start_len = strlen(start);
-+
-+      return (start_len <= strlen(s)) && (0 == strncmp(s, start, start_len));
-+}
-+
-+static size_t mkpath(const char *dir, const char *name, char *out);
 +static simple_git_statuses_t statuses_from_path(const char *path) {
-+      simple_git_statuses_t statuses = { .statuses = NULL, .len = 0 };
-+      git_buf ret = { .ptr = 0, .asize = 0, .size = 0 };
 +      git_repository *repo;
++      git_buf ret = { .ptr = NULL, .asize = 0, .size = 0 };
++      simple_git_statuses_t statuses = { .statuses = NULL, .len = 0 };
 +      git_repository_discover(&ret, path, false, NULL);
 +      git_repository_open(&repo, ret.ptr);
 +      git_buf_dispose(&ret);
 +
 +      if (repo) {
-+              git_status_list *status_list = NULL;
++              char buf[PATH_MAX];
++              const char *workdir = git_repository_workdir(repo);
++              git_status_list *status_list;
++
 +              git_status_list_new(&status_list, repo, NULL);
 +              statuses.len = git_status_list_entrycount(status_list);
 +              statuses.statuses = malloc(statuses.len * sizeof(simple_git_status_t));
-+              const char *workdir = git_repository_workdir(repo);
 +
-+              for (size_t i = 0; i < statuses.len; i ++) {
++              for (size_t i = 0; i < statuses.len; ++i) {
 +                      const git_status_entry *status_ent = git_status_byindex(status_list, i);
 +                      const char *entry_path = status_ent->head_to_index ? status_ent->head_to_index->old_file.path
 +                                      : status_ent->index_to_workdir->old_file.path;
-+                      char *joined = malloc(PATH_MAX * sizeof(char));
-+                      mkpath(workdir, entry_path, joined);
-+                      char *canon_path = realpath(joined, NULL);
-+
-+                      if (canon_path) {
-+                              statuses.statuses[i].path = canon_path;
-+                              free(joined);
-+                      } else
-+                              statuses.statuses[i].path = joined;
 +
++                      xstrsncpy(buf, workdir, xstrlen(workdir));
++                      statuses.statuses[i].path = abspath(entry_path, buf);
 +                      statuses.statuses[i].status = status_ent->status;
 +              }
 +
@@ -147,83 +132,89 @@ index 562622f..96d946a 100644
 +}
 +
 +static void statuses_free(simple_git_statuses_t statuses) {
-+      for (size_t i = 0; i < statuses.len; i ++)
++      for (size_t i = 0; i < statuses.len; ++i)
 +              free(statuses.statuses[i].path);
 +
 +      free(statuses.statuses);
 +}
 +
- static void sigint_handler(int UNUSED(sig))
+ static void resetdircolor(int flags)
  {
-       g_state.interrupt = 1;
-@@ -3794,6 +3907,12 @@ static void printent(const struct entry *ent, uint_t namecols, bool sel)
+       /* Directories are always shown on top, clear the color when moving to first file */
+@@ -3794,6 +3891,11 @@ static void printent(const struct entry *ent, uint_t namecols, bool sel)
 
                if (attrs)
                        attroff(attrs);
 +
-+              if (ent->status_indexed != GIT_COLUMN_STATUS_NONE)
-+                      addch(' ');
-+
-+              print_gitstatus(ent->status_indexed);
-+              print_gitstatus(ent->status_staged);
++              if (git_statuses.len) {
++                      print_gitstatus(ent->status_indexed);
++                      print_gitstatus(ent->status_staged);
++              }
        }
 
        attrs = 0;
-@@ -3809,6 +3928,7 @@ static void printent(const struct entry *ent, uint_t namecols, bool sel)
-                       color_pair = C_MIS;
-               if (color_pair && fcolors[color_pair])
-                       attrs |= COLOR_PAIR(color_pair);
-+
- #ifdef ICONS_ENABLED
-               print_icon(ent, attrs);
- #endif
-@@ -5088,6 +5208,9 @@ static int dentfill(char *path, struct entry **ppdents)
-       struct stat sb_path, sb;
-       DIR *dirp = opendir(path);
+@@ -5119,6 +5221,11 @@ static int dentfill(char *path, struct entry **ppdents)
+               attron(COLOR_PAIR(cfg.curctx + 1));
+       }
 
-+      statuses_free(git_statuses);
-+      git_statuses = statuses_from_path(path);
++      if (git_statuses.len)
++              statuses_free(git_statuses);
++      if (cfg.showdetail)
++              git_statuses = statuses_from_path(path);
 +
-       ndents = 0;
-
-       DPRINTF_S(__func__);
-@@ -5277,6 +5400,26 @@ static int dentfill(char *path, struct entry **ppdents)
-               dentp->gid = sb.st_gid;
+ #if _POSIX_C_SOURCE >= 200112L
+       posix_fadvise(fd, 0, 0, POSIX_FADV_SEQUENTIAL);
+ #endif
+@@ -5316,6 +5423,25 @@ static int dentfill(char *path, struct entry **ppdents)
  #endif
+               }
 
 +              if (git_statuses.len) {
-+                      uint32_t merged_status = 0;
-+                      char joined[PATH_MAX];
-+                      mkpath(path, dentp->name, joined);
-+                      char *real = realpath(joined, NULL);
-+                      char *canon_path = real ? real : joined;
++                      uint merged_status = 0;
++                      char *dentpath = abspath(dentp->name, path);
 +
-+                      for (size_t i = 0; i < git_statuses.len; i ++)
-+                              if (((dentp->mode & S_IFMT) == S_IFDIR) ? starts_with(git_statuses.statuses[i].path, canon_path) :
-+                                              !xstrcmp(git_statuses.statuses[i].path, canon_path) || starts_with(canon_path, git_statuses.statuses[i].path))
-+                                      merged_status |= git_statuses.statuses[i].status;
++                      for (size_t i = 0; i < git_statuses.len; i ++) {
++                              if (dentp->flags & DIR_OR_LINK_TO_DIR) {
++                                      size_t dentlen = xstrlen(dentpath);
++
++                                      if ((dentlen <= xstrlen(git_statuses.statuses[i].path)) && is_prefix(git_statuses.statuses[i].path, dentpath, dentlen))
++                                              merged_status |= git_statuses.statuses[i].status;
++                              } else if (!xstrcmp(git_statuses.statuses[i].path, dentpath))
++                                              merged_status |= git_statuses.statuses[i].status;
++                      }
 +
 +                      dentp->status_indexed = git_get_indexed_status(merged_status);
 +                      dentp->status_staged = git_get_staged_status(merged_status);
-+                      free(real);
-+              } else {
-+                      dentp->status_indexed = GIT_COLUMN_STATUS_NONE;
-+                      dentp->status_staged = GIT_COLUMN_STATUS_NONE;
++                      free(dentpath);
 +              }
 +
-               dentp->flags = S_ISDIR(sb.st_mode) ? 0 : ((sb.st_nlink > 1) ? HARD_LINK : 0);
-               if (entflags) {
-                       dentp->flags |= entflags;
-@@ -7712,6 +7855,8 @@ static void cleanup(void)
+               ++ndents;
+       } while ((dp = readdir(dirp)));
+
+@@ -5834,11 +5960,11 @@ static int adjust_cols(int n)
+ #endif
+       if (cfg.showdetail) {
+               /* Fallback to light mode if less than 35 columns */
+-              if (n < 36)
++              if (n < 38)
+                       cfg.showdetail ^= 1;
+               else {
+                       /* 3 more accounted for below */
+-                      n -= 32;
++                      n -= 34;
+               }
+       }
+
+@@ -7712,6 +7838,8 @@ static void cleanup(void)
                fflush(stdout);
        }
  #endif
-+      statuses_free(git_statuses);
-+      git_libgit2_shutdown();
++  statuses_free(git_statuses);
++  git_libgit2_shutdown();
        free(selpath);
        free(plgpath);
        free(cfgpath);
-@@ -7907,6 +8052,7 @@ int main(int argc, char *argv[])
+@@ -7907,6 +8035,7 @@ int main(int argc, char *argv[])
                return EXIT_FAILURE;
 
        atexit(cleanup);
index 77d4de98dc8e89d4b9ac5c8dd1c99bb2c923c5df..d215fcbe0a5d84cfc94a358fb8d07f9b26f24350 100644 (file)
@@ -3,26 +3,26 @@
 #
 # Dependencies: libgit2
 #
-# Authors: @crides, Luuk van Baaldiff --git a/src/nnn.c b/src/nnn.c
-index d66e5da..5770f1a 100644
+# Authors: @crides, Luuk van Baal
+
+diff --git a/src/nnn.c b/src/nnn.c
+index 22402fa..33be591 100644
 --- a/src/nnn.c
 +++ b/src/nnn.c
-@@ -127,6 +127,8 @@
- #include "qsort.h"
- #endif
-
+@@ -105,6 +105,7 @@
+ #include <wchar.h>
+ #include <pwd.h>
+ #include <grp.h>
 +#include <git2.h>
-+
- /* Macro definitions */
- #define VERSION "4.0"
- #define GENERAL_INFO "BSD 2-Clause\nhttps://github.com/jarun/nnn"
-@@ -245,6 +247,18 @@ typedef unsigned char uchar_t;
+
+ #if !defined(alloca) && defined(__GNUC__)
+ /*
+@@ -244,6 +245,16 @@ typedef unsigned int uint_t;
+ typedef unsigned char uchar_t;
  typedef unsigned short ushort_t;
  typedef unsigned long long ulong_t;
-
 +typedef enum {
-+      GIT_COLUMN_STATUS_NONE = 0,
-+      GIT_COLUMN_STATUS_UNMOD,
++      GIT_COLUMN_STATUS_UNMOD = 0,
 +      GIT_COLUMN_STATUS_NEW,
 +      GIT_COLUMN_STATUS_MODIFIED,
 +      GIT_COLUMN_STATUS_DELETED,
@@ -31,11 +31,10 @@ index d66e5da..5770f1a 100644
 +      GIT_COLUMN_STATUS_IGNORED,
 +      GIT_COLUMN_STATUS_CONFLICTED,
 +} git_column_status_t;
-+
+
  /* STRUCTURES */
 
- /* Directory entry */
-@@ -258,6 +272,8 @@ typedef struct entry {
+@@ -258,6 +269,8 @@ typedef struct entry {
                ulong_t blocks : 40; /* 5 bytes (enough for 512 TiB in 512B blocks allocated) */
                ulong_t nlen : 16; /* 2 bytes (length of file name) */
                ulong_t flags : 8; /* 1 byte (flags specific to the file) */
@@ -44,7 +43,7 @@ index d66e5da..5770f1a 100644
        };
  #ifndef NOUG
        uid_t uid; /* 4 bytes */
-@@ -366,7 +382,18 @@ static struct {
+@@ -366,7 +379,18 @@ static struct {
        ushort_t maxnameln, maxsizeln, maxuidln, maxgidln, maxentln, uidln, gidln, printguid;
  } dtls;
 
@@ -63,9 +62,9 @@ index d66e5da..5770f1a 100644
 
  /* Configuration, contexts */
  static settings cfg = {
-@@ -813,6 +840,92 @@ static void notify_fifo(bool force);
-
- /* Functions */
+@@ -3472,6 +3496,85 @@ static char *get_kv_val(kv *kvarr, char *buf, int key, uchar_t max, uchar_t id)
+       return NULL;
+ }
 
 +static git_column_status_t git_get_indexed_status(const uint32_t status) {
 +      if (status & GIT_STATUS_INDEX_NEW)         return GIT_COLUMN_STATUS_NEW;
@@ -89,7 +88,6 @@ index d66e5da..5770f1a 100644
 +
 +static void print_gitstatus(git_column_status_t status) {
 +      switch (status) {
-+              case GIT_COLUMN_STATUS_NONE:         break;
 +              case GIT_COLUMN_STATUS_UNMOD:        addch('-' | 0); break;
 +              case GIT_COLUMN_STATUS_NEW:          addch('N' | (g_state.oldcolor ? COLOR_PAIR(1) : COLOR_PAIR(C_EXE))); break;
 +              case GIT_COLUMN_STATUS_MODIFIED:     addch('M' | (g_state.oldcolor ? COLOR_PAIR(1) : COLOR_PAIR(4))); break;
@@ -102,41 +100,35 @@ index d66e5da..5770f1a 100644
 +}
 +
 +static bool starts_with(const char *const s, const char *const start) {
-+      const size_t start_len = strlen(start);
++      const size_t start_len = xstrlen(start);
 +
-+      return (start_len <= strlen(s)) && (0 == strncmp(s, start, start_len));
++      return (start_len <= xstrlen(s)) && !strncmp(s, start, start_len);
 +}
 +
-+static size_t mkpath(const char *dir, const char *name, char *out);
 +static simple_git_statuses_t statuses_from_path(const char *path) {
-+      simple_git_statuses_t statuses = { .statuses = NULL, .len = 0 };
-+      git_buf ret = { .ptr = 0, .asize = 0, .size = 0 };
 +      git_repository *repo;
++      git_buf ret = { .ptr = NULL, .asize = 0, .size = 0 };
++      simple_git_statuses_t statuses = { .statuses = NULL, .len = 0 };
 +      git_repository_discover(&ret, path, false, NULL);
 +      git_repository_open(&repo, ret.ptr);
 +      git_buf_dispose(&ret);
 +
 +      if (repo) {
-+              git_status_list *status_list = NULL;
++              char buf[PATH_MAX];
++              const char *workdir = git_repository_workdir(repo);
++              git_status_list *status_list;
++
 +              git_status_list_new(&status_list, repo, NULL);
 +              statuses.len = git_status_list_entrycount(status_list);
 +              statuses.statuses = malloc(statuses.len * sizeof(simple_git_status_t));
-+              const char *workdir = git_repository_workdir(repo);
 +
-+              for (size_t i = 0; i < statuses.len; i ++) {
++              for (size_t i = 0; i < statuses.len; ++i) {
 +                      const git_status_entry *status_ent = git_status_byindex(status_list, i);
 +                      const char *entry_path = status_ent->head_to_index ? status_ent->head_to_index->old_file.path
 +                                      : status_ent->index_to_workdir->old_file.path;
-+                      char *joined = malloc(PATH_MAX * sizeof(char));
-+                      mkpath(workdir, entry_path, joined);
-+                      char *canon_path = realpath(joined, NULL);
-+
-+                      if (canon_path) {
-+                              statuses.statuses[i].path = canon_path;
-+                              free(joined);
-+                      } else
-+                              statuses.statuses[i].path = joined;
 +
++                      xstrsncpy(buf, workdir, xstrlen(workdir));
++                      statuses.statuses[i].path = abspath(entry_path, buf);
 +                      statuses.statuses[i].status = status_ent->status;
 +              }
 +
@@ -147,76 +139,78 @@ index d66e5da..5770f1a 100644
 +}
 +
 +static void statuses_free(simple_git_statuses_t statuses) {
-+      for (size_t i = 0; i < statuses.len; i ++)
++      for (size_t i = 0; i < statuses.len; ++i)
 +              free(statuses.statuses[i].path);
 +
 +      free(statuses.statuses);
 +}
 +
- static void sigint_handler(int UNUSED(sig))
+ static void resetdircolor(int flags)
  {
-       g_state.interrupt = 1;
-@@ -3783,6 +3896,13 @@ static void printent(const struct entry *ent, uint_t namecols, bool sel)
+       /* Directories are always shown on top, clear the color when moving to first file */
+@@ -3783,6 +3886,12 @@ static void printent(const struct entry *ent, uint_t namecols, bool sel)
 
        addch((ent->flags & FILE_SELECTED) ? '+' | A_REVERSE | A_BOLD : ' ');
 
-+      if (cfg.showdetail) {
++      if (cfg.showdetail && git_statuses.len) {
 +              print_gitstatus(ent->status_indexed);
 +              print_gitstatus(ent->status_staged);
-+              if (ent->status_indexed != GIT_COLUMN_STATUS_NONE)
-+                      addch(' ');
++              addch(' ');
 +      }
 +
        if (g_state.oldcolor)
                resetdircolor(ent->flags);
        else {
-@@ -5094,6 +5214,9 @@ static int dentfill(char *path, struct entry **ppdents)
-       struct stat sb_path, sb;
-       DIR *dirp = opendir(path);
+@@ -5125,6 +5234,11 @@ static int dentfill(char *path, struct entry **ppdents)
+               attron(COLOR_PAIR(cfg.curctx + 1));
+       }
 
-+      statuses_free(git_statuses);
-+      git_statuses = statuses_from_path(path);
++      if (git_statuses.len)
++              statuses_free(git_statuses);
++      if (cfg.showdetail)
++              git_statuses = statuses_from_path(path);
 +
-       ndents = 0;
-
-       DPRINTF_S(__func__);
-@@ -5283,6 +5406,26 @@ static int dentfill(char *path, struct entry **ppdents)
-               dentp->gid = sb.st_gid;
+ #if _POSIX_C_SOURCE >= 200112L
+       posix_fadvise(fd, 0, 0, POSIX_FADV_SEQUENTIAL);
+ #endif
+@@ -5322,6 +5436,18 @@ static int dentfill(char *path, struct entry **ppdents)
  #endif
+               }
 
 +              if (git_statuses.len) {
-+                      uint32_t merged_status = 0;
-+                      char joined[PATH_MAX];
-+                      mkpath(path, dentp->name, joined);
-+                      char *real = realpath(joined, NULL);
-+                      char *canon_path = real ? real : joined;
++                      uint merged_status = 0;
 +
 +                      for (size_t i = 0; i < git_statuses.len; i ++)
-+                              if (((dentp->mode & S_IFMT) == S_IFDIR) ? starts_with(git_statuses.statuses[i].path, canon_path) :
-+                                              !xstrcmp(git_statuses.statuses[i].path, canon_path) || starts_with(canon_path, git_statuses.statuses[i].path))
++                              if ((dentp->flags & DIR_OR_LINK_TO_DIR) ? starts_with(git_statuses.statuses[i].path, abspath(dentp->name, path))
++                                              : !xstrcmp(git_statuses.statuses[i].path, abspath(dentp->name, path)))
 +                                      merged_status |= git_statuses.statuses[i].status;
 +
 +                      dentp->status_indexed = git_get_indexed_status(merged_status);
 +                      dentp->status_staged = git_get_staged_status(merged_status);
-+                      free(real);
-+              } else {
-+                      dentp->status_indexed = GIT_COLUMN_STATUS_NONE;
-+                      dentp->status_staged = GIT_COLUMN_STATUS_NONE;
 +              }
 +
-               dentp->flags = S_ISDIR(sb.st_mode) ? 0 : ((sb.st_nlink > 1) ? HARD_LINK : 0);
-               if (entflags) {
-                       dentp->flags |= entflags;
-@@ -7714,6 +7857,8 @@ static void cleanup(void)
+               ++ndents;
+       } while ((dp = readdir(dirp)));
+
+@@ -5972,7 +6098,7 @@ static void redraw(char *path)
+ #endif
+               }
+       }
+-      dtls.maxentln = dtls.maxnameln + dtls.maxsizeln + (dtls.printguid ? (dtls.maxuidln + dtls.maxgidln) : 0) + (g_state.uidgid ? 26 : 23);
++      dtls.maxentln = dtls.maxnameln + dtls.maxsizeln + (dtls.printguid ? (dtls.maxuidln + dtls.maxgidln) : 0) + (g_state.uidgid ? 29 : 26);
+
+       ncols = adjust_cols(ncols);
+
+@@ -7714,6 +7840,8 @@ static void cleanup(void)
                fflush(stdout);
        }
  #endif
-+      statuses_free(git_statuses);
-+      git_libgit2_shutdown();
++  statuses_free(git_statuses);
++  git_libgit2_shutdown();
        free(selpath);
        free(plgpath);
        free(cfgpath);
-@@ -7909,6 +8054,7 @@ int main(int argc, char *argv[])
+@@ -7909,6 +8037,7 @@ int main(int argc, char *argv[])
                return EXIT_FAILURE;
 
        atexit(cleanup);
index 49e614e79463fc4ea621879be5e7b0a5fff10dcf..c4318d42dcc5cf1108510c287fbe83e6a563188e 100644 (file)
@@ -2,8 +2,9 @@
 #              columns when a directory contains different users/groups.
 #
 # Author: Luuk van Baal
+
 diff --git a/src/nnn.c b/src/nnn.c
-index 562622fa..d66e5dac 100644
+index d83d7f6..22402fa 100644
 --- a/src/nnn.c
 +++ b/src/nnn.c
 @@ -362,6 +362,10 @@ typedef struct {
@@ -211,7 +212,7 @@ index 562622fa..d66e5dac 100644
 +#endif
 +              }
 +      }
-+      dtls.maxentln = dtls.maxnameln + dtls.maxsizeln + dtls.maxuidln + dtls.maxgidln + (g_state.uidgid ? 29 : 26);
++      dtls.maxentln = dtls.maxnameln + dtls.maxsizeln + (dtls.printguid ? (dtls.maxuidln + dtls.maxgidln) : 0) + (g_state.uidgid ? 26 : 23);
 +
        ncols = adjust_cols(ncols);