Re: [PATCH] checkpatch: Look for symbolic permissions and suggest octal instead

2016-08-15 Thread Joe Perches
On Tue, 2016-08-02 at 16:39 -0700, Joe Perches wrote: > S_ uses should be avoided where octal is more intelligible. ping? Should CodingStyle and Documentation/filesystems change too? > Signed-off-by: Joe Perches > --- >  scripts/checkpatch.pl | 49

Re: [PATCH] checkpatch: Look for symbolic permissions and suggest octal instead

2016-08-15 Thread Joe Perches
On Tue, 2016-08-02 at 16:39 -0700, Joe Perches wrote: > S_ uses should be avoided where octal is more intelligible. ping? Should CodingStyle and Documentation/filesystems change too? > Signed-off-by: Joe Perches > --- >  scripts/checkpatch.pl | 49

Re: [PATCH] checkpatch: Look for symbolic permissions and suggest octal instead

2016-08-02 Thread Al Viro
On Tue, Aug 02, 2016 at 04:39:24PM -0700, Joe Perches wrote: > S_ uses should be avoided where octal is more intelligible. Oh, for Cthulhu sake! So not only we had been dribbled upon with 1200-odd piles of pointless crap, now we'll be getting yet another set of equally pointless garbage each

Re: [PATCH] checkpatch: Look for symbolic permissions and suggest octal instead

2016-08-02 Thread Al Viro
On Tue, Aug 02, 2016 at 04:39:24PM -0700, Joe Perches wrote: > S_ uses should be avoided where octal is more intelligible. Oh, for Cthulhu sake! So not only we had been dribbled upon with 1200-odd piles of pointless crap, now we'll be getting yet another set of equally pointless garbage each

Re: [PATCH] checkpatch: Look for symbolic permissions and suggest octal instead

2016-08-02 Thread Joe Perches
On Wed, 2016-08-03 at 01:15 +0100, Al Viro wrote: > On Tue, Aug 02, 2016 at 04:39:24PM -0700, Joe Perches wrote: > > > > S_ uses should be avoided where octal is more intelligible. > Oh, for Cthulhu sake!  So not only we had been dribbled upon with 1200-odd > piles of pointless crap, now we'll be

Re: [PATCH] checkpatch: Look for symbolic permissions and suggest octal instead

2016-08-02 Thread Joe Perches
On Wed, 2016-08-03 at 01:15 +0100, Al Viro wrote: > On Tue, Aug 02, 2016 at 04:39:24PM -0700, Joe Perches wrote: > > > > S_ uses should be avoided where octal is more intelligible. > Oh, for Cthulhu sake!  So not only we had been dribbled upon with 1200-odd > piles of pointless crap, now we'll be

[PATCH] checkpatch: Look for symbolic permissions and suggest octal instead

2016-08-02 Thread Joe Perches
S_ uses should be avoided where octal is more intelligible. Signed-off-by: Joe Perches --- scripts/checkpatch.pl | 49 +++-- 1 file changed, 43 insertions(+), 6 deletions(-) diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl

[PATCH] checkpatch: Look for symbolic permissions and suggest octal instead

2016-08-02 Thread Joe Perches
S_ uses should be avoided where octal is more intelligible. Signed-off-by: Joe Perches --- scripts/checkpatch.pl | 49 +++-- 1 file changed, 43 insertions(+), 6 deletions(-) diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index