Re: [PATCH 3/4] vl: Remove useless test in configure_accelerators

2020-01-09 Thread Philippe Mathieu-Daudé
On 1/9/20 3:17 AM, Richard Henderson wrote: The result of g_strsplit is never NULL. Signed-off-by: Richard Henderson --- vl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/vl.c b/vl.c index c9329fe699..887dbfbb5d 100644 --- a/vl.c +++ b/vl.c @@ -2776,7 +2776,7 @@ static

Re: [PATCH 3/4] vl: Remove useless test in configure_accelerators

2020-01-09 Thread Aleksandar Markovic
On Thursday, January 9, 2020, Richard Henderson < richard.hender...@linaro.org> wrote: > The result of g_strsplit is never NULL. > > Signed-off-by: Richard Henderson > --- > vl.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > Reviewed by: Aleksandar Markovic > diff --git a/vl.c

Re: [PATCH 3/4] vl: Remove useless test in configure_accelerators

2020-01-09 Thread Alex Bennée
Richard Henderson writes: > The result of g_strsplit is never NULL. > > Signed-off-by: Richard Henderson Reviewed-by: Alex Bennée > --- > vl.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/vl.c b/vl.c > index c9329fe699..887dbfbb5d 100644 > --- a/vl.c > +++ b/vl.

[PATCH 3/4] vl: Remove useless test in configure_accelerators

2020-01-08 Thread Richard Henderson
The result of g_strsplit is never NULL. Signed-off-by: Richard Henderson --- vl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/vl.c b/vl.c index c9329fe699..887dbfbb5d 100644 --- a/vl.c +++ b/vl.c @@ -2776,7 +2776,7 @@ static void configure_accelerators(const char *prognam