From murdoch at stats.uwo.ca  Fri Oct 20 01:41:50 2006
From: murdoch at stats.uwo.ca (Duncan Murdoch)
Date: Thu, 19 Oct 2006 19:41:50 -0400
Subject: [Rd] bug in rect (PR#9307)
In-Reply-To: <x2y7rbki1b.fsf@turmalin.kubism.ku.dk>
References: <20061019213434.A40F324796@slim.kubism.ku.dk>	<45380390.5010209@stats.uwo.ca>
	<x2y7rbki1b.fsf@turmalin.kubism.ku.dk>
Message-ID: <45380D3E.8010005@stats.uwo.ca>

On 10/19/2006 7:22 PM, Peter Dalgaard wrote:
> Duncan Murdoch <murdoch at stats.uwo.ca> writes:
> 
>> On 10/19/2006 5:34 PM, jgvcqa at rit.edu wrote:
>>> Full_Name: Joe Voelkel
>>> Version: 2.4.0
>>> OS: Windows XP
>>> Submission from: (NULL) (129.21.11.37)
>>>
>>>
>>> Using "border=NA" in the call to rect generates an error. But the help file says
>>> border=NA is acceptable. 
>>>
>>> (The problem is that border=NA evaluates to TRUE for "is.logical(border)" inside
>>> the function. Using rect with border=FALSE works fine)
>> I don't see this.  Could you post some code to illustrate?
> 
> He should, but it is actually plain to see in the code:
> 
>     if (!is.null(density) && !is.null(angle)) {
>         if (is.logical(border)) {
>             if (border)
>                 border <- col
>             else border <- NA
>         }
> 
> so it requires non-default density and angle to bite. The code should
> probably read 
> 
> ...
>    if(!is.na(border) && is.logical(border)) 
> ... 
> 

Right.  Looks like there might be a similar bug lurking in lpolygon and 
lrect in lattice, so I'm cc'ing Deepayan.

Are you fixing rect?

Duncan Murdoch




[Rd] bug in rect (PR#9307)
Duncan Murdoch