On Wed, Jan 13, 2016 at 04:13:07PM +0100, Andrew Jones wrote: > > + assert(dt_get_nr_cells(node, &nac_root, &nsc_root) == 0); > > Sorry the starter code I sent you had statements inside asserts(). > We avoid those now, see commit 18ab6cadf Hmm.. the commit does not appear addressing the described in changelog condition '...if somebody introduces a switch to "#define assert(...) /*nothing*/"...' as to-be-void asserts are not followed by return value checks (even when seemingly possible). If it worth following this pattern? Thanks! _______________________________________________ kvmarm mailing list kvmarm@xxxxxxxxxxxxxxxxxxxxx https://lists.cs.columbia.edu/mailman/listinfo/kvmarm
