From hpages at fhcrc.org  Thu May  2 01:49:49 2013
From: hpages at fhcrc.org (=?ISO-8859-1?Q?Herv=E9_Pag=E8s?=)
Date: Wed, 01 May 2013 16:49:49 -0700
Subject: [Rd] Patch proposal for R style consistency (concerning
	deparse.c)
In-Reply-To: <51816ABC.1070407@gmail.com>
References: <CAErODj85saboM+8SaYKKVVDL6cPeX=LVR=PKwY+FZb2ky_cpBg@mail.gmail.com>
	<5478C197-76AB-481D-A1AA-64DB63D8D122@gmail.com>
	<CAErODj_N+FCjP1zt2H4VHv28o8RDU8sQ-W4qGONkx1gLvC33pg@mail.gmail.com>
	<CANROs4emmr=rGLKWpCQ6z_016mAk7O42=ttA1BLnc9zQ_DTzqQ@mail.gmail.com>
	<51787400.7010305@fhcrc.org> <51787C98.2030106@gmail.com>
	<5178D695.30009@fhcrc.org>
	<CAErODj9nL5SOPCJ1R=NjueZAiG=WJo1V1KwPxbNGJpR1z1bJbg@mail.gmail.com>
	<CAC+N9BWCDacT_cCD3fGAb2EgB+oaQ0qGNq0fkOWsigzZtTJfaA@mail.gmail.com>
	<51816ABC.1070407@gmail.com>
Message-ID: <5181AA1D.1070409@fhcrc.org>

On 05/01/2013 12:19 PM, Duncan Murdoch wrote:
> 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.

Perdon me, but I was not of course suggesting that every substring be
runnable. That would be non-sense. So please, don't make me say what
I didn't say. Thank you!

H.

>
> 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 <pauljohn32 at 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 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

-- 
Herv? Pag?s

Program in Computational Biology
Division of Public Health Sciences
Fred Hutchinson Cancer Research Center
1100 Fairview Ave. N, M1-B514
P.O. Box 19024
Seattle, WA 98109-1024

E-mail: hpages at fhcrc.org
Phone:  (206) 667-5791
Fax:    (206) 667-1319



[Rd] Patch proposal for R style consistency (concerning deparse.c)
=?ISO-8859-1?Q?Herv=E9_Pag=E8s?=