[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] vhost_net: fix misuse of assert()
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH] vhost_net: fix misuse of assert() |
Date: |
Fri, 19 Jun 2015 11:13:40 +0100 |
On 19 June 2015 at 11:07, Michael S. Tsirkin <address@hidden> wrote:
> On Fri, Jun 19, 2015 at 12:00:53PM +0200, Greg Kurz wrote:
>> In case NDEBUG is defined, assert() expands to nothing and
>> vhost_net_set_vnet_endian() doesn't get called...
>>
>> Suggested-by: Thomas Huth <address@hidden>
>> Signed-off-by: Greg Kurz <address@hidden>
>
> Not sure what the point is.
> We don't support building with NDEBUG.
Putting functional behaviour inside an assert() is still a really
bad idea. If you're reading the code you probably skim over the
assert() as not functionally relevant...
-- PMM
- [Qemu-devel] [PATCH] vhost_net: fix misuse of assert(), Greg Kurz, 2015/06/19
- Re: [Qemu-devel] [PATCH] vhost_net: fix misuse of assert(), Michael S. Tsirkin, 2015/06/19
- Re: [Qemu-devel] [PATCH] vhost_net: fix misuse of assert(),
Peter Maydell <=
- Re: [Qemu-devel] [PATCH] vhost_net: fix misuse of assert(), Michael S. Tsirkin, 2015/06/19
- Re: [Qemu-devel] [PATCH] vhost_net: fix misuse of assert(), Markus Armbruster, 2015/06/19
- Re: [Qemu-devel] [PATCH] vhost_net: fix misuse of assert(), Michael S. Tsirkin, 2015/06/19
- Re: [Qemu-devel] [PATCH] vhost_net: fix misuse of assert(), Markus Armbruster, 2015/06/19
- Re: [Qemu-devel] [PATCH] vhost_net: fix misuse of assert(), Michael S. Tsirkin, 2015/06/19
- Re: [Qemu-devel] [PATCH] vhost_net: fix misuse of assert(), Greg Kurz, 2015/06/19