Re: CVS commit: src/sys/dev/sysmon

[Christoph Badura <bad%bsd.de@localhost>]

On Tue, Jun 23, 2015 at 10:41:42AM +0000, Paul Goyette wrote:
> Committed By:	pgoyette
> Date:		Tue Jun 23 10:41:42 UTC 2015
> 
> Modified Files:
> 	src/sys/dev/sysmon: sysmon_envsys_events.c
> 
> Log Message:
> Fix the KASSERT - we want to make sure that _both_ pointers are non-NULL,
> not just that one or the other is non-NULL!

Wouldn't it be better written as:

KASSERT(sme != NULL);
KASSERT(edata != NULL);

Same effect, but when one of the triggers you know immediately which pointer
was NULL.

--chris