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.


i was questioning the change of the message.