R devel - May 2008 - (PR#11281) Bug in R 2.7 for over long lines

On Sat, 2008-05-10 at 11:19 +0100, Prof Brian Ripley
wrote:
> You will see the current code is different, and your 'fix' is not
needed=20
> nor applies in R-devel.
would be nice...
> You failed to provide an example to reproduce the alleged bug, but the=20
well the bug was obvious, I told that I can trigger it and that the
proposed fix fixed it - no need to provide an example.
> issue does seem to be using lines beyond the documented line length.
exactly. one can crash R with too long lines.
> So it would have only affected people who did that ....
or use auto-generated code like e.g. swig produces.
> And generating a new report (PR#11438) was distinctly unfriendly.
I did what I was told, reply and keep the PR#11281 in the subject. I am
sensing another bug.
> If after studing the R FAQ you have a reproducible example in a
> current=20
> version of R (R-devel or R-patched), plus add it to *this* report
> number.
I don't intend to play with devel versions of R, I was just trying to
get swig for R2.7 to work. Sorry that it triggered a bug in R. =EF=BB=BFI will
try R2.7.1 when it is released and report back.

Soeren.

On 5/10/2008 5:20 PM, bugreports at nn7.de wrote:
> On Sat, 2008-05-10 at 11:19 +0100, Prof Brian Ripley wrote:
>> You will see the current code is different, and your 'fix' is
not needed=20
>> nor applies in R-devel.
> 
> would be nice...
> 
>> You failed to provide an example to reproduce the alleged bug, but
the=20
> 
> well the bug was obvious, I told that I can trigger it and that the
> proposed fix fixed it - no need to provide an example.
But it would be helpful to provide an example, so that we can test the 
fix.  As Brian told you, your fix was no good:  it was not against the 
current code.
> 
>> issue does seem to be using lines beyond the documented line length.
> 
> exactly. one can crash R with too long lines.
Then the bug is also in your code, for sending lines that are too long. 
  R shouldn't crash on user error, but "don't do that" is an
appropriate
response.
> 
>> So it would have only affected people who did that ....
> 
> or use auto-generated code like e.g. swig produces.
Then swig should be modified to produce valid code.
> 
>> And generating a new report (PR#11438) was distinctly unfriendly.
> 
> I did what I was told, reply and keep the PR#11281 in the subject. I am
> sensing another bug.
> 
>> If after studing the R FAQ you have a reproducible example in a
>> current=20
>> version of R (R-devel or R-patched), plus add it to *this* report
>> number.
> 
> I don't intend to play with devel versions of R, I was just trying to
> get swig for R2.7 to work. Sorry that it triggered a bug in R. =EF=BB=BFI
w> ill
> try R2.7.1 when it is released and report back.
If you aren't interested in being helpful by testing fixes for your 
code, then I doubt if any of us are going to go out of our way to help 
you with your errors.

Duncan Murdoch
> 
> Soeren.
> 
> ______________________________________________
> R-devel at r-project.org mailing list
> https://stat.ethz.ch/mailman/listinfo/r-devel

On Tue, 2008-05-13 at 09:01 +0200, Martin Maechler
wrote:
> >>>>> "BDR" =3D=3D Prof Brian Ripley <ripley at
stats.ox.ac.uk>
> >>>>>     on Tue, 13 May 2008 07:32:43 +0100 (BST) writes:
>=20
>     BDR> This example does not crash in R 2.7.0, R-patched nor
>     BDR> R-devel (r45677) for me (x86_64 F8 Linux.)  It also
>     BDR> does not crash with the CRAN build of R 2.7.0 on
>     BDR> Windows XP.
>=20
> Neither does it for me in R 2.7.0=20
> on two x86_64 Linux platforms: =20
>    Ubuntu 8.04 and Redhat Enterprise 5.1
=EF=BB=BFsource("http://nn7.de/debugging/Features.R")

crashes here on debian-sid (32bit x86 architecture).

@dirk: could you give it a go on some debian x86 and amd64 machine?

Soeren

On 13 May 2008 at 09:46, Soeren Sonnenburg wrote:
| On Tue, 2008-05-13 at 09:01 +0200, Martin Maechler wrote:
| > >>>>> "BDR" == Prof Brian Ripley <ripley at
stats.ox.ac.uk>
| > >>>>>     on Tue, 13 May 2008 07:32:43 +0100 (BST) writes:
| > 
| >     BDR> This example does not crash in R 2.7.0, R-patched nor
| >     BDR> R-devel (r45677) for me (x86_64 F8 Linux.)  It also
| >     BDR> does not crash with the CRAN build of R 2.7.0 on
| >     BDR> Windows XP.
| > 
| > Neither does it for me in R 2.7.0 
| > on two x86_64 Linux platforms:  
| >    Ubuntu 8.04 and Redhat Enterprise 5.1
| 
| ?source("http://nn7.de/debugging/Features.R")
| 
| crashes here on debian-sid (32bit x86 architecture).
| 
| @dirk: could you give it a go on some debian x86 and amd64 machine?

I don't have easy access to amd64. I could test on x86 for Debian testing,
and in the unstable chroot, but aren't those cases already covered?

Dirk

-- 
Three out of two people have difficulties with fractions.