[comp.sources.d] Nethack patch problem: did anyone else see this

kevin@kosman.UUCP (Kevin O'Gorman) (08/30/87)

After applying the patch, I saw this:

> 	cc -O -c hack.c
> "hack.c", line 89: function has illegal storage class

and sure enough, line 89 has

>	register struct	monst *mtmp, *m_at();

which probably could be


>	register struct	monst *mtmp;
>	extern struct monst *m_at();

since it IS hard to fit a function into a register, even if the function
returns a pointer.  Anyway, hack.c includes hack.h which includes monst.h
which has m_at() declared, so I wonder why this thing was put in at all.

Is noone else's compiler complaining about this?  Is there something that
I am misunderstanding?  (No C guru here, folks, just a user).