From: "Patrick R. Michaud" <[EMAIL PROTECTED]>
   Date: Sun, 13 Apr 2008 11:59:42 -0500

   On Sun, Apr 13, 2008 at 11:34:33AM -0400, Bob Rogers wrote:
   >    From: "James Keenan via RT" <[EMAIL PROTECTED]>
   >    Date: Sun, 13 Apr 2008 05:28:17 -0700
   > 
   >    Of these five files, the only ones that we (or, more to the point, I)
   >    could not safely do right away is languages/BASIC/compiler/compile.pl. 
   >    This is the only instance where it appears to my untrained eyes that the
   >    ops are actually used . . .
   > 
   > Yes, and this code is only used if debugging is enabled.

   And it seems as though only saveall is being used -- there
   doesn't seem to be a corresponding restoreall.

I thought I had seen a restoreall elsewhere, but I can't find it now.

   Perhaps the saveall is for the benefit of the debugger?

You mean pdb?  I see no indication that pdb does anything with
reg_stack.  So this saveall is pointless.

   My suggestion would be to simply comment out 'saveall' with
   a reference to RT#52838 (for anyone who wants to look at it later),
   and then be done with it.

   Pm

Good idea.

                                        -- Bob

Reply via email to