From luke-tierney at uiowa.edu  Wed Jan 12 15:45:33 2011
From: luke-tierney at uiowa.edu (luke-tierney at uiowa.edu)
Date: Wed, 12 Jan 2011 08:45:33 -0600
Subject: [Rd] as.environment.list provides inconsistent results under
 torture
In-Reply-To: <80481BA7-58C3-4C01-8A1A-C2C65EA38F80@gmail.com>
References: <4D2CB092.6000303@r-enthusiasts.com>
	<792C6196-A236-44AC-B734-E38A09791CB4@r-project.org>
	<alpine.DEB.2.00.1101111751380.1812@luke-inspiron>
	<21D8CF42-9C05-4EC0-A6CD-0C697C81D69A@r-project.org>
	<alpine.DEB.2.00.1101120638320.1812@luke-inspiron>
	<80481BA7-58C3-4C01-8A1A-C2C65EA38F80@gmail.com>
Message-ID: <alpine.DEB.2.00.1101120843110.1812@luke-inspiron>

On Wed, 12 Jan 2011, peter dalgaard wrote:

>
> On Jan 12, 2011, at 14:18 , <luke-tierney at uiowa.edu> <luke-tierney at uiowa.edu> wrote:
>>
>> I did do a quick scan of R-devel for this issue with eval and found these:
>>
>>    ./unix/aqua.c:	     eval(LCONS(install("library"),CONS(install("grDevices"),R_NilValue)),R_GlobalEnv);
>>    ./unix/sys-std.c:    infile = PROTECT(eval(lang1(RComp_getFileCompSym), rcompgen_rho));
>>    ./modules/X11/dataentry.c:       newval <- eval(parse(text=newval))
>>    ./main/envir.c:			 return(eval(lang4(install("list2env"), arg,
>>    ./gnuwin32/dataentry.c:       newval <- eval(parse(text=newval))
>>
>> I'll fix them in the next couple of days if no one else gets there
>> first (but I'm not set up to test the aqua or gnuwin32 ones).
>>
>
> You had me perplexed there for a while: "<-" assignment in C code??? However, those lines occur inside comments, so at least gnuwin32 is not to worry about.

:-) I guess I would have noticed that when I started trying to fix things ...

>
> As you look around those parts, you might also find some superfluous PROTECTs inserted by me some time in the last century while trying to chase down the mysterious crashes that we had. (They eventually turned out to be caused by parser lookahead breaking protection stack discipline, fixed by UNPROTECT_PTR() etc.). I lost track a bit in the frenzy and I never got around to backing out everything.

I think for now the "if it ain't broke don't fix it" principle applies
to those, unless you have spare time on your hands ...

Best,

luke


-- 
Luke Tierney
Statistics and Actuarial Science
Ralph E. Wareham Professor of Mathematical Sciences
University of Iowa                  Phone:             319-335-3386
Department of Statistics and        Fax:               319-335-3017
    Actuarial Science
241 Schaeffer Hall                  email:      luke at stat.uiowa.edu
Iowa City, IA 52242                 WWW:  http://www.stat.uiowa.edu



[Rd] as.environment.list provides inconsistent results under torture
luke-tierney at uiowa.edu