diff mbox

[dpdk-dev] virtio: fix missing curly braces

Message ID 20160713092419.14425-1-viktorin@rehivetech.com (mailing list archive)
State Rejected, archived
Headers

Commit Message

Jan Viktorin July 13, 2016, 9:24 a.m. UTC 
  GCC 6 is complaining and seems to be correct here.

virtio_user_ethdev.c:345:2: error:
	this ‘if’ clause does not guard... [-Werror=misleading-indentation]
  if (rte_kvargs_count(kvlist, VIRTIO_USER_ARG_PATH) == 1)
  ^~

virtio_user_ethdev.c:348:3: note:
	...this statement, but the latter is misleadingly indented
	as if it is guarded by the ‘if’
   if (ret < 0) {

Fixes: 404bd6bfe360 ("net/virtio-user: fix return value not checked")
Signed-off-by: Jan Viktorin <viktorin@rehivetech.com>
---
 drivers/net/virtio/virtio_user_ethdev.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Maxime Coquelin July 13, 2016, 9:27 a.m. UTC | #1 
Hi Jan,

On 07/13/2016 11:24 AM, Jan Viktorin wrote:
> GCC 6 is complaining and seems to be correct here.
>
> virtio_user_ethdev.c:345:2: error:
> 	this ‘if’ clause does not guard... [-Werror=misleading-indentation]
>    if (rte_kvargs_count(kvlist, VIRTIO_USER_ARG_PATH) == 1)
>    ^~
>
> virtio_user_ethdev.c:348:3: note:
> 	...this statement, but the latter is misleadingly indented
> 	as if it is guarded by the ‘if’
>     if (ret < 0) {
>
> Fixes: 404bd6bfe360 ("net/virtio-user: fix return value not checked")
> Signed-off-by: Jan Viktorin <viktorin@rehivetech.com>
> ---
>
I already fixed it yesterday:
http://dpdk.org/dev/patchwork/patch/14780/

Thanks,
Maxime
Jan Viktorin July 13, 2016, 5:42 p.m. UTC | #2 
On Wed, 13 Jul 2016 11:27:18 +0200
Maxime Coquelin <maxime.coquelin@redhat.com> wrote:

> Hi Jan,
> 
> On 07/13/2016 11:24 AM, Jan Viktorin wrote:
> > GCC 6 is complaining and seems to be correct here.
> >
> > virtio_user_ethdev.c:345:2: error:
> > 	this ‘if’ clause does not guard... [-Werror=misleading-indentation]
> >    if (rte_kvargs_count(kvlist, VIRTIO_USER_ARG_PATH) == 1)
> >    ^~
> >
> > virtio_user_ethdev.c:348:3: note:
> > 	...this statement, but the latter is misleadingly indented
> > 	as if it is guarded by the ‘if’
> >     if (ret < 0) {
> >
> > Fixes: 404bd6bfe360 ("net/virtio-user: fix return value not checked")
> > Signed-off-by: Jan Viktorin <viktorin@rehivetech.com>
> > ---
> >  
> I already fixed it yesterday:
> http://dpdk.org/dev/patchwork/patch/14780/

Sorry, I didn't find it quickly. My fault. Thanks.

Jan

> 
> Thanks,
> Maxime
Jan Viktorin July 13, 2016, 5:43 p.m. UTC | #3 
On Wed, 13 Jul 2016 11:27:18 +0200
Maxime Coquelin <maxime.coquelin@redhat.com> wrote:

> Hi Jan,
> 
> On 07/13/2016 11:24 AM, Jan Viktorin wrote:
> > GCC 6 is complaining and seems to be correct here.
> >
> > virtio_user_ethdev.c:345:2: error:
> > 	this ‘if’ clause does not guard... [-Werror=misleading-indentation]
> >    if (rte_kvargs_count(kvlist, VIRTIO_USER_ARG_PATH) == 1)
> >    ^~
> >
> > virtio_user_ethdev.c:348:3: note:
> > 	...this statement, but the latter is misleadingly indented
> > 	as if it is guarded by the ‘if’
> >     if (ret < 0) {
> >
> > Fixes: 404bd6bfe360 ("net/virtio-user: fix return value not checked")
> > Signed-off-by: Jan Viktorin <viktorin@rehivetech.com>
> > ---
> >  
> I already fixed it yesterday:
> http://dpdk.org/dev/patchwork/patch/14780/

Sorry, I didn't find it quickly. My fault. Thanks.

Jan

> 
> Thanks,
> Maxime
Maxime Coquelin July 13, 2016, 8:19 p.m. UTC | #4 
On 07/13/2016 07:42 PM, Jan Viktorin wrote:
> On Wed, 13 Jul 2016 11:27:18 +0200
> Maxime Coquelin <maxime.coquelin@redhat.com> wrote:
>
>> Hi Jan,
>>
>> On 07/13/2016 11:24 AM, Jan Viktorin wrote:
>>> GCC 6 is complaining and seems to be correct here.
>>>
>>> virtio_user_ethdev.c:345:2: error:
>>> 	this ‘if’ clause does not guard... [-Werror=misleading-indentation]
>>>    if (rte_kvargs_count(kvlist, VIRTIO_USER_ARG_PATH) == 1)
>>>    ^~
>>>
>>> virtio_user_ethdev.c:348:3: note:
>>> 	...this statement, but the latter is misleadingly indented
>>> 	as if it is guarded by the ‘if’
>>>     if (ret < 0) {
>>>
>>> Fixes: 404bd6bfe360 ("net/virtio-user: fix return value not checked")
>>> Signed-off-by: Jan Viktorin <viktorin@rehivetech.com>
>>> ---
>>>
>> I already fixed it yesterday:
>> http://dpdk.org/dev/patchwork/patch/14780/
>
> Sorry, I didn't find it quickly. My fault. Thanks.
Oh no problem! Better having two fixes than none :)

Thanks,
Maxime
diff mbox

Patch

diff --git a/drivers/net/virtio/virtio_user_ethdev.c b/drivers/net/virtio/virtio_user_ethdev.c
index 17d5848..1b903f3 100644
--- a/drivers/net/virtio/virtio_user_ethdev.c
+++ b/drivers/net/virtio/virtio_user_ethdev.c
@@ -342,7 +342,7 @@  virtio_user_pmd_devinit(const char *name, const char *params)
 		goto end;
 	}
 
-	if (rte_kvargs_count(kvlist, VIRTIO_USER_ARG_PATH) == 1)
+	if (rte_kvargs_count(kvlist, VIRTIO_USER_ARG_PATH) == 1) {
 		ret = rte_kvargs_process(kvlist, VIRTIO_USER_ARG_PATH,
 					 &get_string_arg, &path);
 		if (ret < 0) {
@@ -350,7 +350,7 @@  virtio_user_pmd_devinit(const char *name, const char *params)
 				     VIRTIO_USER_ARG_PATH);
 			goto end;
 		}
-	else {
+	} else {
 		PMD_INIT_LOG(ERR, "arg %s is mandatory for virtio-user\n",
 			  VIRTIO_USER_ARG_QUEUE_SIZE);
 		goto end;