> in theory, i see the problem. allthough i'd suggest to break lines after
> commas only, so the only problematic case would be a comma followed by space
> inside a quote:
>
> c(" ", ' ', "', '", "\" \"", "\\ ")
>
> but still, even if we ignored this for the time being, i fail to see the
> benefit that the current state
>
> c(" ", ' ', "', '
>
> has over the proposed
>
> c(" ", ' ', "',
> '", "\" \"",
> "\\ ")
>
> sure this isn't perfect either, but at least you get to see the rest of the
> vector. so i'd consider this an improvement, nonetheless. it should work in
> almost all of the cases, because it would only apply to options which are too
> long anyway, and what are the odds that these also have a quoted comma+space
> exactly in the problematic place?
Basically, I don't want to add code that I know doesn't work -
obviously, I'd prefer not to already have code that doesn't work, but
I want to make it better, not worse.
Hadley
--
Assistant Professor / Dobelman Family Junior Chair
Department of Statistics / Rice University
http://had.co.nz/
_______________________________________________
Roxygen-devel mailing list
Roxygen-devel@lists.r-forge.r-project.org
https://lists.r-forge.r-project.org/cgi-bin/mailman/listinfo/roxygen-devel
