Re: [Qemu-devel] [PATCH 07/10] pseries: Clean up error handling in spapr_rtas_register()

2016-01-20 Thread Thomas Huth
On 20.01.2016 06:53, Alexey Kardashevskiy wrote: > On 01/20/2016 03:53 PM, Eric Blake wrote: >> On 01/19/2016 05:21 PM, Alexey Kardashevskiy wrote: >> You could drop the redundant () while touching this, as in: >>> >>> >>> Seriously? Why? I personally find it really annoying (but I stay silent

Re: [Qemu-devel] [PATCH 07/10] pseries: Clean up error handling in spapr_rtas_register()

2016-01-20 Thread Markus Armbruster
Alexey Kardashevskiy writes: > On 01/20/2016 06:18 PM, Markus Armbruster wrote: >> Eric Blake writes: >> >>> On 01/19/2016 05:21 PM, Alexey Kardashevskiy wrote: >>> > You could drop the redundant () while touching this, as in: Seriously? Why? I personally find it really annoyi

Re: [Qemu-devel] [PATCH 07/10] pseries: Clean up error handling in spapr_rtas_register()

2016-01-20 Thread Alexey Kardashevskiy
On 01/20/2016 06:18 PM, Markus Armbruster wrote: Eric Blake writes: On 01/19/2016 05:21 PM, Alexey Kardashevskiy wrote: You could drop the redundant () while touching this, as in: Seriously? Why? I personally find it really annoying (but I stay silent) when people omit braces in cases lik

Re: [Qemu-devel] [PATCH 07/10] pseries: Clean up error handling in spapr_rtas_register()

2016-01-19 Thread Markus Armbruster
Eric Blake writes: > On 01/19/2016 05:21 PM, Alexey Kardashevskiy wrote: > >>> You could drop the redundant () while touching this, as in: >> >> >> Seriously? Why? I personally find it really annoying (but I stay silent) >> when people omit braces in cases like this. >> >> >>> assert(token >=

Re: [Qemu-devel] [PATCH 07/10] pseries: Clean up error handling in spapr_rtas_register()

2016-01-19 Thread Alexey Kardashevskiy
On 01/20/2016 03:53 PM, Eric Blake wrote: On 01/19/2016 05:21 PM, Alexey Kardashevskiy wrote: You could drop the redundant () while touching this, as in: Seriously? Why? I personally find it really annoying (but I stay silent) when people omit braces in cases like this. assert(token >= RT

Re: [Qemu-devel] [PATCH 07/10] pseries: Clean up error handling in spapr_rtas_register()

2016-01-19 Thread Eric Blake
On 01/19/2016 05:21 PM, Alexey Kardashevskiy wrote: >> You could drop the redundant () while touching this, as in: > > > Seriously? Why? I personally find it really annoying (but I stay silent) > when people omit braces in cases like this. > > >> assert(token >= RTAS_TOKEN_BASE && token < RTAS

Re: [Qemu-devel] [PATCH 07/10] pseries: Clean up error handling in spapr_rtas_register()

2016-01-19 Thread Alexey Kardashevskiy
On 01/20/2016 09:58 AM, Eric Blake wrote: On 01/15/2016 05:00 AM, David Gibson wrote: The errors detected in this function necessarily indicate bugs in the rest of the qemu code, rather than an external or configuration problem. So, a simple assert() is more appropriate than any more complex er

Re: [Qemu-devel] [PATCH 07/10] pseries: Clean up error handling in spapr_rtas_register()

2016-01-19 Thread Eric Blake
On 01/15/2016 05:00 AM, David Gibson wrote: > The errors detected in this function necessarily indicate bugs in the rest > of the qemu code, rather than an external or configuration problem. > > So, a simple assert() is more appropriate than any more complex error > reporting. > > Signed-off-by:

[Qemu-devel] [PATCH 07/10] pseries: Clean up error handling in spapr_rtas_register()

2016-01-15 Thread David Gibson
The errors detected in this function necessarily indicate bugs in the rest of the qemu code, rather than an external or configuration problem. So, a simple assert() is more appropriate than any more complex error reporting. Signed-off-by: David Gibson --- hw/ppc/spapr_rtas.c | 12 +++-