🚀 go-pugleaf

There are some run-time errors in GCC 2.1, cp-decl.c, (detected with Saber-C).

In cp-decl.c, line 578, there is the field declaration
(in struct binding_level):

	#ifdef PARANOID
	    unsigned char depth;
	#endif

And in line 711 (function pop_binding_level), the sign of this unsigned
char is reversed:

	#ifdef PARANOID
	    level->depth *= -1;
	#endif

What should that be, a substraction (level->depth -= 1;)?

And shouldn't there be `#if PARANOID' instead of `#ifdef PARANOID'?
In cp-tree.h, line 26, PARANOID is set to 0:

	#ifndef PARANOID
	#define PARANOID 0
	#endif

There is a more serious error with the use of finish_decl (cp-decl.c,
line 5068).  The function is supposed to be passed four parameters.
The fourth parameter, `need_pop', tells whether obstacks should be
popped.  In ALL the calls to finish_decl from cp-parse.y, only three
parameters are passed.

--Gerald