Message ID | 1456016178-4560-1-git-send-email-tsimonq2@ubuntu.com (mailing list archive) |
---|---|
State | Superseded |
Headers | show |
Better? On 02/20/2016 06:56 PM, Simon Quigley wrote: > checkpatch.pl reported a warning of over 80 characters on line 1833 > > Adjusted to put &extref and &offset on a different line > > Signed-off-by: Simon Quigley <tsimonq2@ubuntu.com> > --- > fs/btrfs/backref.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/fs/btrfs/backref.c b/fs/btrfs/backref.c > index f6dac40..956fffa 100644 > --- a/fs/btrfs/backref.c > +++ b/fs/btrfs/backref.c > @@ -1830,8 +1830,8 @@ static int iterate_inode_extrefs(u64 inum, struct btrfs_root *fs_root, > unsigned long ptr; > > while (1) { > - ret = btrfs_find_one_extref(fs_root, inum, offset, path, &extref, > - &offset); > + ret = btrfs_find_one_extref(fs_root, inum, offset, path, > + &extref, &offset); > if (ret < 0) > break; > if (ret) { > -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Sat, 2016-02-20 at 18:57 -0600, Simon Quigley wrote: > Better? No, not really. The alignment should be to the open parenthesis as I wrote in the first reply. > On 02/20/2016 06:56 PM, Simon Quigley wrote: > > checkpatch.pl reported a warning of over 80 characters on line 1833 > > > > Adjusted to put &extref and &offset on a different line > > > > Signed-off-by: Simon Quigley <tsimonq2@ubuntu.com> > > --- > > fs/btrfs/backref.c | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/fs/btrfs/backref.c b/fs/btrfs/backref.c > > index f6dac40..956fffa 100644 > > --- a/fs/btrfs/backref.c > > +++ b/fs/btrfs/backref.c > > @@ -1830,8 +1830,8 @@ static int iterate_inode_extrefs(u64 inum, > > struct btrfs_root *fs_root, > > unsigned long ptr; > > > > while (1) { > > - ret = btrfs_find_one_extref(fs_root, inum, offset, > > path, &extref, > > - &offset); > > + ret = btrfs_find_one_extref(fs_root, inum, offset, > > path, > > + &extref, > > &offset); > > if (ret < 0) > > break; > > if (ret) { > > -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/fs/btrfs/backref.c b/fs/btrfs/backref.c index f6dac40..956fffa 100644 --- a/fs/btrfs/backref.c +++ b/fs/btrfs/backref.c @@ -1830,8 +1830,8 @@ static int iterate_inode_extrefs(u64 inum, struct btrfs_root *fs_root, unsigned long ptr; while (1) { - ret = btrfs_find_one_extref(fs_root, inum, offset, path, &extref, - &offset); + ret = btrfs_find_one_extref(fs_root, inum, offset, path, + &extref, &offset); if (ret < 0) break; if (ret) {
checkpatch.pl reported a warning of over 80 characters on line 1833 Adjusted to put &extref and &offset on a different line Signed-off-by: Simon Quigley <tsimonq2@ubuntu.com> --- fs/btrfs/backref.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)