6 Oct
2013
6 Oct
'13
1:41 p.m.
On Sat, Oct 05, 2013 at 12:16:07PM +0200, Alexander Chemeris wrote:
#define SUBSCR_PUT(sub) \
- sub->net = &dummy_net; \
- subscr_put(sub);
- if (sub) { \
sub->net = &dummy_net; \subscr_put(sub); \- }
I don't like this semantic change. Either the tests expects a valid subscriber or it doesn't. It is not a "maybe" this query will result in a look up.