On Fri, Feb 28, 2014 at 09:14:03AM +0200, Alan Barrett wrote:
> On Thu, 27 Feb 2014, Joerg Sonnenberger wrote:
> >Modified Files:
> >     src/sys/dev/pci: if_ti.c
> >
> >Log Message:
> >Remove impossible checks.
> 
> If new bugs are introduced in the future, then these tests might no
> longer be impossible.  So I'd prefer to change them to KASSERT or
> KDASSERT rather than deleting them.

Given the nature of this checks and the failure mode of such changes, I
don't see a point in the assert.

Joerg

Reply via email to