[Help-smalltalk] Re: STCompiler ignores current namespace change for var lookup in evals

[Stephen Compall]

Paolo Bonzini wrote:
> It is.  The attached patch should fix the problem, but I have not
> tested it, nor applied it yet.  If you get back to me before 2.3, I
> will include it (very safe, applies only to compiler), but otherwise I
> won't.

While this fixes STCompiler>>#evaluate:parser:, 
STFileInParser>>#evaluate: doesn't use that method, and can't unless you 
want to expand the protocol of Behavior>>#compilerClass's answer.  OTOH, 
I think it would be less than ideal to add the current namespace to the 
symbol table in compile:asMethodOf:classified:parser:.

Using STCompiler>>#evaluate:parser: in the valueWithUnwind'd block would 
also subtly change STFileInParser>>#evaluate:'s meaning -- because 
compiler errors wouldn't cause exceptions that could escape the 
evaluate: context.

(tested; same as before)

--
Stephen Compall
http://scompall.nocandysw.com/blog
##smalltalk,#gnu-smalltalk on Freenode IRC