Re: [Rd] segfault with POSIXlt zone=NULL zone=""

2016-12-06 Thread frederik
Thanks for the reply, Duncan. It looks like Martin can commit a patch faster than I can open a bug report... Frederick On Tue, Dec 06, 2016 at 12:39:32PM -0500, Duncan Murdoch wrote: > I agree this is a bug; R should never segfault. I wouldn't call it a high > priority one, since you can avoid

Re: [Rd] segfault with POSIXlt zone=NULL zone=""

2016-12-06 Thread Martin Maechler
> Joshua Ulrich > on Tue, 6 Dec 2016 09:51:16 -0600 writes: > On Tue, Dec 6, 2016 at 6:37 AM, wrote: >> Hi all, >> >> I ran into a segfault while playing with dates. >> >> $ R --no-init-file >> ... >> >

Re: [Rd] segfault with POSIXlt zone=NULL zone=""

2016-12-06 Thread Joshua Ulrich
On Tue, Dec 6, 2016 at 10:30 AM, wrote: > Hi Joshua, > > Thank you for minimizing my test case. > >> > Hope I'm not doing something illegal... >> > >> You are. You're changing the internal structure of a POSIXlt object >> by re-ordering the list elements. You should not

Re: [Rd] segfault with POSIXlt zone=NULL zone=""

2016-12-06 Thread frederik
Hi Joshua, Thank you for minimizing my test case. > > Hope I'm not doing something illegal... > > > You are. You're changing the internal structure of a POSIXlt object > by re-ordering the list elements. You should not expect a malformed > POSIXlt object to behave as if it's correctly formed.

Re: [Rd] segfault with POSIXlt zone=NULL zone=""

2016-12-06 Thread Joshua Ulrich
On Tue, Dec 6, 2016 at 6:37 AM, wrote: > Hi all, > > I ran into a segfault while playing with dates. > > $ R --no-init-file > ... > > library(lubridate); d=as.POSIXlt(floor_date(Sys.time(),"year")); > d$zone=NULL; d$zone=""; d > If you're asking about a bug in R,

[Rd] segfault with POSIXlt zone=NULL zone=""

2016-12-06 Thread frederik
Hi all, I ran into a segfault while playing with dates. $ R --no-init-file ... > library(lubridate); d=as.POSIXlt(floor_date(Sys.time(),"year")); d$zone=NULL; d$zone=""; d Attaching package: ‘lubridate’ The following object is masked from ‘package:base’: date