* Re: btrfs-progs: v3, move out print in cmd_df to another function
@ 2013-09-01 6:15 Wang Shilong
2013-09-02 2:58 ` Anand Jain
0 siblings, 1 reply; 3+ messages in thread
From: Wang Shilong @ 2013-09-01 6:15 UTC (permalink / raw)
To: Anand Jain; +Cc: linux-btrfs
Hello, Using checkpatch.pl, i get the following warnings(errors):
ERROR: "foo * bar" should be "foo *bar"
#37: FILE: cmds-filesystem.c:47:
+static char * group_type_str(u64 flag)
ERROR: "foo * bar" should be "foo *bar"
#69: FILE: cmds-filesystem.c:63:
+static char * group_profile_str(u64 flag)
total: 2 errors, 0 warnings, 229 lines checked
patch has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
Thanks, Wang
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: btrfs-progs: v3, move out print in cmd_df to another function
2013-09-01 6:15 btrfs-progs: v3, move out print in cmd_df to another function Wang Shilong
@ 2013-09-02 2:58 ` Anand Jain
2013-09-02 3:00 ` Wang Shilong
0 siblings, 1 reply; 3+ messages in thread
From: Anand Jain @ 2013-09-02 2:58 UTC (permalink / raw)
To: Wang Shilong; +Cc: linux-btrfs
Wang, kindly note that this was fixed in v4.
Anand
On 09/01/2013 02:15 PM, Wang Shilong wrote:
> Hello, Using checkpatch.pl, i get the following warnings(errors):
> ERROR: "foo * bar" should be "foo *bar"
> #37: FILE: cmds-filesystem.c:47:
> +static char * group_type_str(u64 flag)
>
> ERROR: "foo * bar" should be "foo *bar"
> #69: FILE: cmds-filesystem.c:63:
> +static char * group_profile_str(u64 flag)
>
> total: 2 errors, 0 warnings, 229 lines checked
>
> patch has style problems, please review. If any of these errors
> are false positives report them to the maintainer, see
> CHECKPATCH in MAINTAINERS.
>
> Thanks, Wang
> --
> 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
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: btrfs-progs: v3, move out print in cmd_df to another function
2013-09-02 2:58 ` Anand Jain
@ 2013-09-02 3:00 ` Wang Shilong
0 siblings, 0 replies; 3+ messages in thread
From: Wang Shilong @ 2013-09-02 3:00 UTC (permalink / raw)
To: Anand Jain; +Cc: Wang Shilong, linux-btrfs
On 09/02/2013 10:58 AM, Anand Jain wrote:
>
> Wang, kindly note that this was fixed in v4.
Yeah...This is sent by my shell script by accident, I have fixed it and
it won't send the repeated message again.
Thanks for reminding .. ^_^
>
> Anand
>
>
> On 09/01/2013 02:15 PM, Wang Shilong wrote:
>> Hello, Using checkpatch.pl, i get the following warnings(errors):
>> ERROR: "foo * bar" should be "foo *bar"
>> #37: FILE: cmds-filesystem.c:47:
>> +static char * group_type_str(u64 flag)
>>
>> ERROR: "foo * bar" should be "foo *bar"
>> #69: FILE: cmds-filesystem.c:63:
>> +static char * group_profile_str(u64 flag)
>>
>> total: 2 errors, 0 warnings, 229 lines checked
>>
>> patch has style problems, please review. If any of these errors
>> are false positives report them to the maintainer, see
>> CHECKPATCH in MAINTAINERS.
>>
>> Thanks, Wang
>> --
>> 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
>>
> --
> 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
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2013-09-02 3:02 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-09-01 6:15 btrfs-progs: v3, move out print in cmd_df to another function Wang Shilong
2013-09-02 2:58 ` Anand Jain
2013-09-02 3:00 ` Wang Shilong
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).