Re: [PATCH 1/4] statx: Add a system call to make enhanced file info available [ver #3]

From: David Howells
Date: Thu Nov 24 2016 - 02:30:36 EST


Andreas Dilger <adilger@xxxxxxxxx> wrote:

> > + case S_IFCHR: printf(" character special file\n"); ft = 'c'; break;
>
> This will overflow 80 columns. Could use just "character special"?
>
> > + case S_IFDIR: printf(" directory\n"); ft = 'd'; break;
> > + case S_IFBLK: printf(" block special file\n"); ft = 'b'; break;
> > + case S_IFREG: printf(" regular file\n"); ft = '-'; break;
> > + case S_IFLNK: printf(" symbolic link\n"); ft = 'l'; break;
>
> Having these all as single-word items would make it easier to parse for
> test programs. Same with "IO Block:" -> "Blocksize:". Not a huge deal.

Try doing "stat /dev/null".

David