Re: [PATCH] mm: memory: fixed a coding style issue

From: Michal Hocko
Date: Tue Jan 16 2018 - 02:47:03 EST


On Mon 15-01-18 19:17:12, Robert Donald Rickett wrote:
> This is a patch to the memory.c file that fixes the
> "ERROR: code indent should use tabs where possible"
> found by the checkpatch.pl tool.

Is this really worth it? The code is not any better readable and it just
adds a churn to the history and makes life of anybody using git blame
slightly more harder. So what is the benefit?

> Signed-off-by: Robert Donald Rickett <robert.rickett@xxxxxxxxx>
> ---
> mm/memory.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/mm/memory.c b/mm/memory.c
> index ca5674cbaff2..e9f6e58aa77c 100644
> --- a/mm/memory.c
> +++ b/mm/memory.c
> @@ -1663,7 +1663,7 @@ int zap_vma_ptes(struct vm_area_struct *vma, unsigned long address,
> unsigned long size)
> {
> if (address < vma->vm_start || address + size > vma->vm_end ||
> - !(vma->vm_flags & VM_PFNMAP))
> + !(vma->vm_flags & VM_PFNMAP))
> return -1;
> zap_page_range_single(vma, address, size, NULL);
> return 0;
> --
> 2.14.1

--
Michal Hocko
SUSE Labs