On Tue, Apr 22, 2008 at 5:46 AM, Will Coleda <[EMAIL PROTECTED]> wrote:
> On Tue, Apr 22, 2008 at 2:52 AM, Patrick R. Michaud <[EMAIL PROTECTED]> wrote:
> >
>
>
> > Tonight I resolved RT#52838 and RT#52840, eliminating the
> > saveall/restoreall
> > opcodes, the register stack, and fixing up tests and other items
> > to avoid save/restore/etc. opcodes.
> >
> > I've also created a "userstack" branch where I've got the details
> > for removing the rest of stack.ops . I've already implemented
> > and tested it (it's pretty straightforward), but since DEPRECATED.pod
> > says that these opcodes are to be removed "post 0.7.0" it's
> > too early to commit these changes to trunk.
> >
> > One of the interesting features of removing stack.ops is that
> > we no longer need the STACK_ENTRY_INT, STACK_ENTRY_FLOAT,
> > or STACK_ENTRY_STRING types in src/stacks.c, as those were being
> > used only by the save/restore opcodes. Should we see about removing
> > these entry types from the code, or should we leave them there in
> > case we come up with a need for them in the future?
>
> I'd remove them. There's always subversion if we need to get them back later.
>
+1
> Thanks for the cleanup work!
>
+1
~jerry
