Re: [kvm-unit-tests PATCH 19/18] don't embed code inside asserts

2015-11-10 Thread Paolo Bonzini
On 09/11/2015 21:57, Andrew Jones wrote: > assert() is classically a macro which could also be disabled, so if > somebody introduces a switch to "#define assert(...) /*nothing*/" in > the future, we'd lose code. > > Suggested-by: Thomas Huth > Signed-off-by: Andrew Jones

[kvm-unit-tests PATCH 19/18] don't embed code inside asserts

2015-11-09 Thread Andrew Jones
assert() is classically a macro which could also be disabled, so if somebody introduces a switch to "#define assert(...) /*nothing*/" in the future, we'd lose code. Suggested-by: Thomas Huth Signed-off-by: Andrew Jones --- lib/arm/setup.c | 19