[Freeciv-Dev] Re: (PR#9870) bad assert in aitools.h

["Gregory Berkolaiko" <Gregory.Berkolaiko@xxxxxxxxxxxxx>]

<URL: http://rt.freeciv.org/Ticket/Display.html?id=9870 >

On Sun, 29 Aug 2004, Jason Short wrote:

> <URL: http://rt.freeciv.org/Ticket/Display.html?id=9870 >
> 
> assert(find_unit_by_id(punit->id))
> 
> but pointer checks need to be explicit on some platforms in assert 
> (presumably because their implementation of assert sucks).
> 
> So it should be a != NULL check.

isn't this assert a bit crazy?
because if find_unit_by_id fails, it means that there is no such unit and 
punit is pointing to some pretty much random data!