[Roxygen-devel] fix for aliases

[Hadley Wickham]

Would you mind adding this to
https://github.com/klutometis/roxygen/issues/24?  That way it won't
get lost in my inbox.
Hadley

On Thu, Sep 1, 2011 at 12:39 PM, meik michalke
<meik.michalke at uni-duesseldorf.de> wrote:
> hi,
>
> as R CMD check revealed to me, the values of \alias{} must not be quoted.
> otherwise you run into warnings regarding undocumented stuff.
>
> here's another fix, in two patches:
>
> <patch1>
> --- roclet-namespace.R.orig     2011-08-29 23:52:59.648650390 +0200
> +++ roclet-namespace.R  2011-09-01 19:25:54.228446954 +0200
> @@ -182,9 +182,15 @@
>   unlist(lapply(matches, f, tag = tag, all = partitum), use.names = FALSE)
>  }
>
> -words <- function(x) {
> -  quote_if_needed(str_split(str_trim(x), "\\s+")[[1]])
> +words <- function(x, quote=TRUE) {
> +  x.trimmed <- str_split(str_trim(x), "\\s+")[[1]]
> +  if(isTRUE(quote)){
> +    return(quote_if_needed(x.trimmed))
> +  } else {
> +    return(x.trimmed)
> +  }
>  }
> +
>  is.syntactic <- function(x) make.names(x) == x
>  has.quotes <- function(x) str_detect(x, "'|\"")
>  quote_if_needed <- function(x) {
> </patch1>
>
> <patch2>
> --- roclet-rd.R.orig    2011-07-28 20:59:24.000000000 +0200
> +++ roclet-rd.R 2011-09-01 19:27:26.976672899 +0200
> @@ -333,7 +333,7 @@
>   add_tag(rd, process_description(partitum, base_path))
>
>   add_tag(rd, process_had_tag(partitum, 'aliases', function(tag, param) {
> -      new_tag('alias', words(param))
> +      new_tag('alias', words(param, quote=FALSE))
>     }))
>   add_tag(rd, process.usage(partitum))
>   add_tag(rd, process.arguments(partitum))
> </patch2>
>
>
> viele grüße :: m.eik
>
> --
> dipl. psych. meik michalke
> institut f"ur experimentelle psychologie
> abt. f"ur diagnostik und differentielle psychologie
> heinrich-heine-universit"at d-40204 d"usseldorf
>



-- 
Assistant Professor / Dobelman Family Junior Chair
Department of Statistics / Rice University
http://had.co.nz/