On 01/05/2013 1:34 PM, Tim Triche, Jr. wrote:
+1 to having runnable code emitted

It does emit runnable code, which is why Herve's complaint was nonsense. It doesn't emit code of which every substring is runnable.

Duncan Murdoch


patch seems to work nicely, hopefully R-core will agree to apply it to HEAD



On Wed, May 1, 2013 at 9:45 AM, Paul Johnson <pauljoh...@gmail.com> wrote:

> Whoa.
>
> Don't let my valuable suggestion get lost.
>
>  I want "} else {".  Yihue wants "} else {".  And I have not heard anybody
> say they prefer the other way, unless you interpret Duncan's comment
> "that's nonsense" as a blanket defense of the status quo. But I don't think
> he meant that.  This is a matter of style consistency and avoidance of new
> R-user confusion and error.
>
> After reading the help for "if", I don't see how anybody can argue against
> this.  Good R code has this style:
>
> } else {
>
> and not
>
> }
>  else
>
> because the latter fails if it is run line-by-line.  While trying to teach
> people how to write R programs, it would be nice if the output of
> print.function was consistent with the good way, the way that is actually
> practiced in the R source code itself. This is a major source of new
> programmer confusion. Its very tough to explain and teach.
>
> pj
> --
> Paul E. Johnson
> Professor, Political Science      Assoc. Director
> 1541 Lilac Lane, Room 504      Center for Research Methods
> University of Kansas                 University of Kansas
> http://pj.freefaculty.org               http://quant.ku.edu
>
>         [[alternative HTML version deleted]]
>
> ______________________________________________
> R-devel@r-project.org mailing list
> https://stat.ethz.ch/mailman/listinfo/r-devel
>




______________________________________________
R-devel@r-project.org mailing list
https://stat.ethz.ch/mailman/listinfo/r-devel

Reply via email to