"Michael S. Tsirkin" <[email protected]> writes: > On Fri, Jun 19, 2015 at 11:13:40AM +0100, Peter Maydell wrote: >> On 19 June 2015 at 11:07, Michael S. Tsirkin <[email protected]> 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 <[email protected]> >> >> Signed-off-by: Greg Kurz <[email protected]> >> > >> > 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 > > I can apply this if commit log explains it's a readability > enhancement, not a bugfix.
Easy: vhost_net: fix misuse of assert() In case NDEBUG is defined, assert() expands to nothing and vhost_net_set_vnet_endian() doesn't get called... Suggested-by: Thomas Huth <[email protected]> Signed-off-by: Greg Kurz <[email protected]> We don't support building with NDEBUG, but 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... Signed-off-by: Michael S. Tsirkin" <[email protected]>
