diff mbox

[6/6] linux-generic: branches of if both reqire parens

Message ID 1440516404-4177-6-git-send-email-mike.holmes@linaro.org
State New
Headers show

Commit Message

Mike Holmes Aug. 25, 2015, 3:26 p.m. UTC
Signed-off-by: Mike Holmes <mike.holmes@linaro.org>
---
 platform/linux-generic/odp_schedule.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Maxim Uvarov Aug. 25, 2015, 7:09 p.m. UTC | #1
short description should have layer name and right spelling.

linux-generic: scheduler: branches of if both require parents

Might be add some prefix that we change only style, and no any actual 
changes. It might
be easy for other platform implementers to understand what is that patch.

Maxim.

On 08/25/15 18:26, Mike Holmes wrote:
> Signed-off-by: Mike Holmes <mike.holmes@linaro.org>
> ---
>   platform/linux-generic/odp_schedule.c | 3 ++-
>   1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/platform/linux-generic/odp_schedule.c b/platform/linux-generic/odp_schedule.c
> index fc5677c..8ca4765 100644
> --- a/platform/linux-generic/odp_schedule.c
> +++ b/platform/linux-generic/odp_schedule.c
> @@ -198,8 +198,9 @@ int odp_schedule_term_global(void)
>   
>   					if (num > 0)
>   						ODP_ERR("Queue not empty\n");
> -				} else
> +				} else {
>   					odp_buffer_free(buf);
> +				}
>   			}
>   
>   			if (odp_queue_destroy(pri_q)) {
diff mbox

Patch

diff --git a/platform/linux-generic/odp_schedule.c b/platform/linux-generic/odp_schedule.c
index fc5677c..8ca4765 100644
--- a/platform/linux-generic/odp_schedule.c
+++ b/platform/linux-generic/odp_schedule.c
@@ -198,8 +198,9 @@  int odp_schedule_term_global(void)
 
 					if (num > 0)
 						ODP_ERR("Queue not empty\n");
-				} else
+				} else {
 					odp_buffer_free(buf);
+				}
 			}
 
 			if (odp_queue_destroy(pri_q)) {