code review 6454072: doc: clarify that gccgo is a GCC front-end

doc: clarify that gccgo is a GCC front-end

[Mike.Rosset, 2012-07-31 17:31:46]

Hello golang-dev@googlegroups.com, I'd like you to review this change to https://code.google.com/p/go

[r, 2012-08-01 01:51:27]

I don't see what's wrong with the current text. -rob 

[Mike.Rosset, 2012-08-01 08:14:16]

On 2012/08/01 01:51:27, r wrote: > I don't see what's wrong with the current text. > > -rob Hello, it is a small change I understand. But if you are selectively reading the faq with no prior knowledge that gccgo is a GCC frontend. Then it can be read as gccgo is C++ front-end, which is technically not accurate.

[iant2, 2012-08-01 13:25:45]

On Wed, Aug 1, 2012 at 1:14 AM, <Mike.Rosset@gmail.com> wrote: > > Hello, it is a small change I understand. But if you are selectively > reading the faq with no prior knowledge that gccgo is a GCC frontend. > Then it can be read as gccgo is C++ front-end, which is technically not > accurate. How about "Gccgo has a front-end written in C++, with a recursive descent parser, coupled...." Ian 

[Mike.Rosset, 2012-08-01 13:38:24]

On 2012/08/01 13:25:45, iant2 wrote: > On Wed, Aug 1, 2012 at 1:14 AM, <mailto:Mike.Rosset@gmail.com> wrote: > > > > Hello, it is a small change I understand. But if you are selectively > > reading the faq with no prior knowledge that gccgo is a GCC frontend. > > Then it can be read as gccgo is C++ front-end, which is technically not > > accurate. > > How about "Gccgo has a front-end written in C++, with a recursive > descent parser, coupled...." > > Ian ah good suggestion thanks.

[Mike.Rosset, 2012-08-01 13:42:41]

Hello golang-dev@googlegroups.com, r@golang.org, iant@google.com (cc: golang-dev@googlegroups.com), Please take another look.

[iant, 2012-08-01 13:44:23]

LGTM with change but wait for r http://codereview.appspot.com/6454072/diff/7001/doc/go_faq.html File doc/go_faq.html (right): http://codereview.appspot.com/6454072/diff/7001/doc/go_faq.html#newcode1347 doc/go_faq.html:1347: <code>Gccgo</code> has a front-end written in C++, with a recursive descent parser coupled to the s/parser coupled/parser, coupled/

[r, 2013-03-12 23:42:49]

LGTM although i would s/front-end/front end/ since it's a noun phrase. sorry for the delay

[iant, 2013-03-13 00:13:01]

*** Submitted as https://code.google.com/p/go/source/detail?r=f3ca7d5b1b0b *** doc: clarify that gccgo is a GCC front-end R=golang-dev, r, iant, iant CC=golang-dev https://codereview.appspot.com/6454072 Committer: Ian Lance Taylor <iant@golang.org>

[remyoudompheng, 2013-07-21 19:48:07]

R=close