Re: [PATCH v1 10/59] chardev/char-win.c: remove 'fail' label in win_chr_serial_init()

2020-01-06 Thread Philippe Mathieu-Daudé
On 1/6/20 7:23 PM, Daniel Henrique Barboza wrote: The 'fail' label is a simply call to 'return -1' on error. Remove it and do the return call instead. CC: Marc-André Lureau CC: Paolo Bonzini Signed-off-by: Daniel Henrique Barboza Reviewed-by: Philippe Mathieu-Daudé --- chardev/char-win

Re: [PATCH v1 10/59] chardev/char-win.c: remove 'fail' label in win_chr_serial_init()

2020-01-06 Thread Marc-André Lureau
On Mon, Jan 6, 2020 at 10:25 PM Daniel Henrique Barboza wrote: > > The 'fail' label is a simply call to 'return -1' on error. Remove > it and do the return call instead. > > CC: Marc-André Lureau > CC: Paolo Bonzini > Signed-off-by: Daniel Henrique Barboza Reviewed-by: Marc-André Lureau > --

[PATCH v1 10/59] chardev/char-win.c: remove 'fail' label in win_chr_serial_init()

2020-01-06 Thread Daniel Henrique Barboza
The 'fail' label is a simply call to 'return -1' on error. Remove it and do the return call instead. CC: Marc-André Lureau CC: Paolo Bonzini Signed-off-by: Daniel Henrique Barboza --- chardev/char-win.c | 19 --- 1 file changed, 8 insertions(+), 11 deletions(-) diff --git a/ch