cleaned up the file with checkpatch
Signed-off-by: Philippe Loctaux
---
fs/btrfs/file.c | 24 ++--
1 file changed, 14 insertions(+), 10 deletions(-)
diff --git a/fs/btrfs/file.c b/fs/btrfs/file.c
index 3a14c87..d131b8d 100644
--- a/fs/btrfs/file.c
+++ b/fs/btrfs/file.c
Hello!
I'm new to the mailing list and btrfs in general (I've been using it for
two weeks on my new arch install) and I'd like to contribute to the code :)
I know how to work w/ git and patches, I just wanted to know which
git repo I need to clone to start contributing :)
I went on the kernel wi
Hello!
I'm new to the mailing list and btrfs in general (I've been using it for
two weeks on my new arch install) and I'd like to contribute to the code :)
I know how to work w/ git and patches, I just wanted to know which
git repo I need to clone to start contributing :)
I went on the kernel w
Fixed a coding style issue.
Signed-off-by: Philippe Loctaux
---
fs/btrfs/async-thread.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/fs/btrfs/async-thread.c b/fs/btrfs/async-thread.c
index 5fb60ea..7914dd2 100644
--- a/fs/btrfs/async-thread.c
+++ b/fs/btrfs/async
On Sun, Feb 21, 2016 at 03:08:02PM -0600, Simon Quigley wrote:
> I thought I proposed this exact patch...hmm...
Yes you're right, I wasn't subscribed at the mailing list yet.
The maintainer will probably see it's a duplicate and he'll just ignore
that patch.
Makes the comment block start with /* on a separate line, and end
with */ on a separate line as well.
Signed-off-by: Philippe Loctaux
---
fs/btrfs/check-integrity.c | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/fs/btrfs/check-integrity.c b/fs/btrfs/check-integrity.c
Makes the comment blocks start with /* on separate lines, and end
with */ on separate lines as well.
Signed-off-by: Philippe Loctaux
---
fs/btrfs/check-integrity.c | 30 ++
1 file changed, 22 insertions(+), 8 deletions(-)
diff --git a/fs/btrfs/check-integrity.c b/fs
Added line after variable declaration, fixing checkpatch warning.
Signed-off-by: Philippe Loctaux
---
fs/btrfs/compression.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/fs/btrfs/compression.c b/fs/btrfs/compression.c
index 3346cd8..5194b6f 100644
--- a/fs/btrfs/compression.c
+++ b/fs
Added lines after variable declarations, fixing 22 checkpatch warnings.
Signed-off-by: Philippe Loctaux
---
fs/btrfs/ctree.c | 23 +++
1 file changed, 23 insertions(+)
diff --git a/fs/btrfs/ctree.c b/fs/btrfs/ctree.c
index 769e0ff..c9fd42d 100644
--- a/fs/btrfs/ctree.c
Use tabs instead of spaces, fixing checkpatch error.
Signed-off-by: Philippe Loctaux
---
fs/btrfs/delayed-inode.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/fs/btrfs/delayed-inode.c b/fs/btrfs/delayed-inode.c
index b57daa8..2e25aa54 100644
--- a/fs/btrfs/delayed-inode.c
Is there no need of additional blank line here particulary
or in all lines that I changed?
--
Philippe Loctaux
p...@philippeloctaux.com
On Sun, Feb 21, 2016 at 03:53:04PM -0800, Joe Perches wrote:
> On Mon, 2016-02-22 at 00:46 +0100, Philippe Loctaux wrote:
> > Added lines after
Use tabs instead of spaces, fixing 3 checkpatch errors.
Signed-off-by: Philippe Loctaux
---
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 cbb7dbf..b89a8c6 100644
--- a/fs/btrfs/dev-replace.c
Hi,
I'm really sorry, but I don't understand what you're trying to mean.
Could you simplify your sentence please (since I'm not english native)?
I'd really apreciate that, thanks :)
--
Philippe Loctaux
p...@philippeloctaux.com
On Sun, Feb 21, 2016 at 04:06:03PM -0800, Jo
Makes the comment blocks start with /* on separate lines, and end
with */ on separate lines as well.
Signed-off-by: Philippe Loctaux
---
fs/btrfs/dev-replace.c | 12
1 file changed, 8 insertions(+), 4 deletions(-)
diff --git a/fs/btrfs/dev-replace.c b/fs/btrfs/dev-replace.c
index
Oh, I see now :)
I'll try your changes and tell if they work or not :)
--
Philippe Loctaux
p...@philippeloctaux.com
On Sun, Feb 21, 2016 at 04:37:54PM -0800, Joe Perches wrote:
> On Mon, 2016-02-22 at 00:26 +0100, Philippe Loctaux wrote:
> > Added line after variable decla
progs.git devel
Allright, I'll work from these repos!
Is it fine if I work from Linus's repo?
Will my patches get rejected if I do so?
--
Philippe Loctaux
p...@philippeloctaux.com
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a messag
Oh okay, thanks :)
--
Philippe Loctaux
p...@philippeloctaux.com
On Sun, Feb 21, 2016 at 04:44:14PM -0800, Joe Perches wrote:
> On Mon, 2016-02-22 at 01:31 +0100, Philippe Loctaux wrote:
> > Hi,
> > I'm really sorry, but I don't understand what you're trying to mea
Added line after variable declaration, fixing checkpatch warning.
Signed-off-by: Philippe Loctaux
---
fs/btrfs/dir-item.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/fs/btrfs/dir-item.c b/fs/btrfs/dir-item.c
index 1752625..70c39a3 100644
--- a/fs/btrfs/dir-item.c
+++ b/fs/btrfs/dir
But for huge modifications, it's recommended to use integration branch, as
> the final pull request is sent to btrfs maintainer Chris, not Linus.
For the moment I'll send small ones, I don't wanna get dirty too fast :P
Thanks for your quick reply!
--
Philippe Loctaux
p...@philippeloc
Makes the comment block start with /* on a separate line, and end
with */ on a separate line as well.
Signed-off-by: Philippe Loctaux
---
fs/btrfs/disk-io.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/fs/btrfs/disk-io.c b/fs/btrfs/disk-io.c
index f642b30..2545d01
Added spaces required around the equal sign, fixing a checkpatch error.
Signed-off-by: Philippe Loctaux
---
fs/btrfs/disk-io.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/fs/btrfs/disk-io.c b/fs/btrfs/disk-io.c
index 84cbf16..f642b30 100644
--- a/fs/btrfs/disk-io.c
+++ b
Fixed a coding style issue.
Signed-off-by: Philippe Loctaux
---
fs/btrfs/disk-io.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/fs/btrfs/disk-io.c b/fs/btrfs/disk-io.c
index 4545e2e..84cbf16 100644
--- a/fs/btrfs/disk-io.c
+++ b/fs/btrfs/disk-io.c
@@ -1908,11
eas#Cleanup_projects
Thanks, I'll take a look and see what I can do!
--
Philippe Loctaux
p...@philippeloctaux.com
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
he new persons (hopefully)
won't make the same mistakes that I did :)
Is there a way we can put this onto the main kernel wiki so that almost
anyone can read that, not just the btrfs people?
--
Philippe Loctaux
p...@philippeloctaux.com
--
To unsubscribe from this list: send the line "unsubs
pace "issues" to fix in btrfs alone.
I agree with you, I won't spend all my time fixing style/whitespace
issues, now that I know what I can do, thanks for that :)
--
Philippe Loctaux
p...@philippeloctaux.com
--
To unsubscribe from this list: send the line "unsubscribe linux-b
25 matches
Mail list logo
