cameronbracken / pgfSweave

Quality graphics and speedy compilation with Sweave
http://r-forge.r-project.org/projects/pgfsweave/
29 stars 6 forks source link

don't reformat the code when keep.source=TRUE #23

Closed yihui closed 13 years ago

yihui commented 13 years ago

If the user wants to keep the source, I think he really wants the source code not to be touched, so the logic seems to be keep.source=TRUE implies tidy=FALSE. Does this make sense?

cameronbracken commented 13 years ago

I think since both options are FALSE by default, which takes precedence is sort of arbitrary but I feel like the user should be able to recognize when the options are clashing and make the change to use only keep.source=TRUE. Is this a problem for you?

yihui commented 13 years ago

this is not a big problem; I just want to save a little bit typing efforts :)

cameronbracken commented 13 years ago

Closing this for now.