Re: [PATCH v3 0/5] Fix some style problems in contrib

kindly ping, the link of this patch set is:http://patchwork.ozlabs.org/project/qemu-devel/cover/20210118031004.1662363-1-zhouyang...@huawei.com/ On 2021/1/18 11:09, zhouyang wrote: > v3 -> v2 > add Cc > > zhouyang (5): > contrib: Don't use '#' flag of printf format >

[PATCH v3 1/5] contrib: Don't use '#' flag of printf format

I am reading contrib related code and found some style problems while check the code using checkpatch.pl. This commit fixs the misuse of '#' flag of printf format Signed-off-by: zhouyang --- contrib/plugins/hotblocks.c | 2 +- contrib/plugins/hotpages.c | 2 +- contrib/plugins/howvec.c| 2

[PATCH v3 5/5] contrib: Open brace '{' following struct go on the same line

I found some style problems whil check the code using checkpatch.pl. This commit fixs the issue below: ERROR: that open brace { should be on the previous line Signed-off-by: zhouyang --- contrib/plugins/howvec.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/contrib

[PATCH v3 2/5] contrib: Fix some code style problems, ERROR: "foo * bar" should be "foo *bar"

I am reading contrib related code and found some style problems while check the code using checkpatch.pl. This commit fixs the issue below: ERROR: "foo * bar" should be "foo *bar" Signed-off-by: zhouyang --- contrib/plugins/howvec.c | 2 +- 1 file changed, 1 insertion(+),

[PATCH v3 0/5] Fix some style problems in contrib

v3 -> v2 add Cc zhouyang (5): contrib: Don't use '#' flag of printf format contrib: Fix some code style problems, ERROR: "foo * bar" should be "foo *bar" contrib: Add spaces around operator contrib: space required after that ',' contrib: Open brace

[PATCH v3 3/5] contrib: Add spaces around operator

I am reading contrib related code and found some style problems while check the code using checkpatch.pl. This commit fixs the issue below: ERROR: spaces required around that '*' Signed-off-by: zhouyang --- contrib/ivshmem-server/main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion

[PATCH v3 4/5] contrib: space required after that ','

I am reading contrib related code and found some style problems while check the code using checkpatch.pl. This commit fixs the issue below: ERROR: space required after that ',' Signed-off-by: zhouyang --- contrib/plugins/howvec.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions

Ping: [PATCH v2 0/5] Fix some style problems in contrib

Hi Peter, Friendly ping again, It's been more than a month since I submitted the patch,did I miss any response? On 2020/12/31 10:26, zhouyang (T) wrote: > kindly ping > > On 2020/12/17 11:44, zhouyang (T) wrote: >> kindly ping >> >>> v1 -> v2: >>> Chan

Re: [PATCH v2 0/5] Fix some style problems in contrib

kindly ping On 2020/12/17 11:44, zhouyang (T) wrote: > kindly ping > >> v1 -> v2: >> Changed the "From:" and "Signed-off-by:" lines from "zhouyang (T)" >> to my real name "zhouyang". >> >> I found some style pro

Re:[PATCH v2 0/5] Fix some style problems in contrib

kindly ping >v1 -> v2: >Changed the "From:" and "Signed-off-by:" lines from "zhouyang (T)" >to my real name "zhouyang". > >I found some style problems while check the code using checkpatch.pl >and fixed them, please review. >

[PATCH] fsdev: open brace '{' following struct go on the same line

I found some style problems while check the code using checkpatch.pl. This commit fixs the issue below: ERROR: open brace '{' following struct go on the same line Signed-off-by: zhouyang --- fsdev/9p-marshal.h | 12 fsdev/file-op-9p.h | 3 +-- 2 files changed, 5 insertions(+), 10

[PATCH v2 5/5] contrib: Open brace '{' following struct go on the same line

I found some style problems whil check the code using checkpatch.pl. This commit fixs the issue below: ERROR: that open brace { should be on the previous line Signed-off-by: zhouyang --- contrib/plugins/howvec.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/contrib

[PATCH v2 3/5] contrib: Add spaces around operator

I am reading contrib related code and found some style problems while check the code using checkpatch.pl. This commit fixs the issue below: ERROR: spaces required around that '*' Signed-off-by: zhouyang --- contrib/ivshmem-server/main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion

[PATCH v2 0/5] Fix some style problems in contrib

v1 -> v2: Changed the "From:" and "Signed-off-by:" lines from "zhouyang (T)" to my real name "zhouyang". I found some style problems while check the code using checkpatch.pl and fixed them, please review. zhouyang (5): contrib: Don't use '#' fla

[PATCH v2 1/5] contrib: Don't use '#' flag of printf format

I am reading contrib related code and found some style problems while check the code using checkpatch.pl. This commit fixs the misuse of '#' flag of printf format Signed-off-by: zhouyang --- contrib/plugins/hotblocks.c | 2 +- contrib/plugins/hotpages.c | 2 +- contrib/plugins/howvec.c| 2

[PATCH v2 4/5] contrib: space required after that ','

I am reading contrib related code and found some style problems while check the code using checkpatch.pl. This commit fixs the issue below: ERROR: space required after that ',' Signed-off-by: zhouyang --- contrib/plugins/howvec.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions

[PATCH v2 2/5] contrib: Fix some code style problems, ERROR: "foo * bar" should be "foo *bar"

I am reading contrib related code and found some style problems while check the code using checkpatch.pl. This commit fixs the issue below: ERROR: "foo * bar" should be "foo *bar" Signed-off-by: zhouyang --- contrib/plugins/howvec.c | 2 +- 1 file changed, 1 insertion(+),

[PATCH 1/5] contrib: Don't use '#' flag of printf format

I found some style problems while check the code using checkpatch.pl. This commit fixs the misuse of '#' flag of printf format Signed-off-by: zhouyang (T) --- contrib/plugins/hotblocks.c | 2 +- contrib/plugins/hotpages.c | 2 +- contrib/plugins/howvec.c| 2 +- contrib/plugins/lockstep.c

[PATCH 4/5] contrib: space required after that ','

I found some style problems while check the code using checkpatch.pl. This commit fixs the issue below: ERROR: space required after that ',' Signed-off-by: zhouyang (T) --- contrib/plugins/howvec.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/contrib/plugins

[PATCH 2/5] contrib: Fix some code style problems, ERROR: "foo * bar" should be "foo *bar"

I found some style problems while check the code using checkpatch.pl. This commit fixs the issue below: ERROR: "foo * bar" should be "foo *bar" Signed-off-by: zhouyang (T) --- contrib/plugins/howvec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/contri

[PATCH 3/5] contrib: Add spaces around operator

I found some style problems while check the code using checkpatch.pl. This commit fixs the issue below: ERROR: spaces required around that '*' Signed-off-by: zhouyang (T) --- contrib/ivshmem-server/main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/contrib/ivshmem-server

[PATCH 0/5] Fix some style problems in contrib

I found some style problems while check the code using checkpatch.pl and fixed them, please review. zhouyang (T) (5): contrib: Don't use '#' flag of printf format contrib: Fix some code style problems, ERROR: "foo * bar" should be "foo *bar" contrib: Add sp

[PATCH 5/5] contrib: Open brace '{' following struct go on the same line

I found some style problems while check the code using checkpatch.pl. This commit fixs the issue below: ERROR: that open brace { should be on the previous line Signed-off-by: zhouyang (T) --- contrib/plugins/howvec.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/contrib