This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Re: A patch to constify gcc.c (Really, summarizing remaining warnings)


 > From: Richard Henderson <rth@cygnus.com>
 > 
 > On Sun, Mar 14, 1999 at 07:28:00AM -0500, Kaveh R. Ghazi wrote:
 > > 	However, gotos and jmp stuff make my head hurt, so please
 > > examine this with a careful eye to make sure I got everything right. :-) 
 > > (I did a successful bootstrap on Irix6.)
 > 
 > It looks correct.
 > 
 > > 	Note, I made one change to the original do_float_handler you
 > > posted which was to call set_float_handler(NULL) in the case where
 > > setjmp returned a value > 0.  This makes sure all exit points of
 > > do_float_handler reset the handler back to zero. 
 > 
 > If you examine the signal handler where longjmp is called, you'll 
 > see that it (in effect) resets the handler there. 
 > 
 > Actually, if all this was cleaned up so that set_float_handler was
 > not called from anywhere but do_float_handler, some of this could
 > definitely be cleaned up a tad.  So I'd not worry about your extra
 > change until then.
 > 
 > > 	Is this okay to install?
 > 
 > Yes.
 > r~

	Thanks, installed.

		--Kaveh

PS: I'll be sending patches for the other "might be clobbered" warnings
shortly. 

--
Kaveh R. Ghazi			Engagement Manager / Project Services
ghazi@caip.rutgers.edu		Qwest Internet Solutions


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]