[Rd] Small performance bug in [.Date

Hadley Wickham h.wickham at gmail.com
Tue Nov 21 14:23:11 CET 2017


> Yes, at least so it looks in  src/main/objects.c
>
> Also, we had a very similar change a while ago :
> ------------------------------------------------------------------------
> r65926 | luke | 2014-06-12 15:54:38 +0200 (Thu, 12. Jun 2014) | 2 Zeilen
> Geänderte Pfade:
>    M src/library/base/R/datetime.R
>
> Commented out class(x) <- NULL in [.POSIXct and [[.POSICct.
> ------------------------------------------------------------------------
>
> and we never seemed to have followed up in a systematic manner
> finding other places where this happens and could be
> eliminated.  I see about half a dozen examples in
> base/R/dates.R  alone and am trying to find more in other places.
>
> [maybe this used to be necessary for very early different
>  versions of NextMethod() which were not yet optimized using  .Class etc]

Thanks for making the fix!

Hadley

-- 
http://hadley.nz



More information about the R-devel mailing list