lkml.org 
[lkml]   [2013]   [Nov]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] FS: BTRFS: fixed coding style issues
On Mon, Nov 04, 2013 at 03:27:38PM +0200, Aldo Iljazi wrote:
> Fixed three coding style issues. Replaced spaces with tabs.
>
> Signed-off-by: Aldo Iljazi <mail@aldo.io>
> ---
> fs/btrfs/dev-replace.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/fs/btrfs/dev-replace.c b/fs/btrfs/dev-replace.c
> index 9efb94e..b2fe609 100644
> --- a/fs/btrfs/dev-replace.c
> +++ b/fs/btrfs/dev-replace.c
> @@ -377,7 +377,7 @@ int btrfs_dev_replace_start(struct btrfs_root *root,
> printk_in_rcu(KERN_INFO
> "btrfs: dev_replace from %s (devid %llu) to %s) started\n",
> src_device->missing ? "<missing disk>" :
> - rcu_str_deref(src_device->name),
> + rcu_str_deref(src_device->name),

What's the change here? I don't think we need to fix whitespace, this
makes searching in patch history more boring, namely in case where the
code looks exactly the same before and after the patch.

The style issues should be best fixed when the patch is about to be
merged, doing it later like this is kind of not welcome, speaking for
myself. There are lots of opportunities to do real code cleanups.


david


\
 
 \ /
  Last update: 2013-11-12 15:01    [W:0.062 / U:0.088 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site