matthew green wrote:
>     
>    -#ifdef DIAGNOSTIC
>    -		panic("hashinit: invalid table type");
>    -#else
>    +		panic("invalid table type");
>     		return NULL;
> 
> why remove the function name?

It panics a bit faster this way.

It's just a typo caused by changing things forward and backward. The DIAGNOSTIC
was removed on purpose. I see no reason to crash with a NULL pointer dereference
later instead.

-- 
Christian