Re: [PATCH v4 2/4] xfs: Fix WS in xfs_isilocked() calls

[Dave Chinner <david@fromorbit.com>]

On Tue, Feb 11, 2020 at 11:10:16PM +0100, Pavel Reichl wrote:
> Signed-off-by: Pavel Reichl <preichl@redhat.com>
> ---
>  fs/xfs/libxfs/xfs_bmap.c | 2 +-
>  fs/xfs/xfs_file.c        | 3 ++-
>  fs/xfs/xfs_inode.c       | 6 +++---
>  fs/xfs/xfs_qm.c          | 2 +-
>  4 files changed, 7 insertions(+), 6 deletions(-)

Hi Pavel,

I don't know what this patch does from reading the subject line,
and the commit message doesn't tell me, either. :)

IOWs, please don't use abbreviations in the subject line because not
everyone understands what the abbreviation you used means. This
makes it hard to read the git history once the patch is committed.
Similarly, subjects that say "Fix the frobnozzle" can be better
written as something like:

xfs: clean up whitespace in xfs_isilocked() calls

So that the output of 'git log --oneline fs/xfs' it's totally
obvious what the commit is doing from the summary.  "Fix the
frobnozzle" is ambiguous because "fix" can mean many things, hence
to find out you need to look at the specific commit in more detail
and that slows down code and history searches....

Cheers,

Dave.
-- 
Dave Chinner
david@fromorbit.com