Re: [Rd] Suggested tidying up

From: Uwe Ligges <ligges_at_statistik.tu-dortmund.de>
Date: Tue, 01 Jun 2010 22:19:07 +0200

On 31.05.2010 23:13, Vincent Goulet wrote:
> Package grDevices has a function cm() to convert from inches to centimeters. Its definition is:
>
> cm<- function(x) 2.54*x
>
> As far as I can tell, the function is not used anywhere in the R sources (I grepped for "cm(", "cm)" and ", cm".) I did not check for all packages on CRAN, though.

I found it easier to remember cm() rather than 2.54.

By the way, before deprecating (rather than at once removing) a function: have you also grepped through all CRAN and BioC packages? And have you grepped for "\"cm\"", "'cm'" and other occurrences? cm may be called in a call(), do.call() or other construct, hence the real test is to remove the cm function, rebuild R and run the checks and run checks on all at least all 3000 packages from CRAN + BioC.

Best wishes,
Uwe Ligges

> May I propose to remove this apparently useless function from the sources?
>
> [I stubbled upon this function because my package actuar also has a cm() function that masks the one from grDevices.]
>
>
> Dr. Vincent Goulet
> Full Professor
> École d'actuariat, Université Laval, Québec
> vincent.goulet@act.ulaval.ca http://vgoulet.act.ulaval.ca
>
> ______________________________________________
> R-devel_at_r-project.org mailing list
> https://stat.ethz.ch/mailman/listinfo/r-devel



R-devel_at_r-project.org mailing list
https://stat.ethz.ch/mailman/listinfo/r-devel Received on Tue 01 Jun 2010 - 20:26:20 GMT

Archive maintained by Robert King, hosted by the discipline of statistics at the University of Newcastle, Australia.
Archive generated by hypermail 2.2.0, at Wed 02 Jun 2010 - 17:20:53 GMT.

Mailing list information is available at https://stat.ethz.ch/mailman/listinfo/r-devel. Please read the posting guide before posting to the list.

list of date sections of archive