Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(560)

Issue 6848048: code review 6848048: 2012/goforc: Slides for "Go for C programmers" talk. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years, 5 months ago by gri
Modified:
11 years, 5 months ago
Reviewers:
minux1, iant2
CC:
r, golang-dev
Visibility:
Public.

Description

2012/goforc: Slides for "Go for C programmers" talk.

Patch Set 1 #

Patch Set 2 : diff -r b8eea95e35a3 https://gri%40golang.org@code.google.com/p/go.talks/ #

Patch Set 3 : diff -r 44382830d29e https://code.google.com/p/go.talks #

Patch Set 4 : diff -r 44382830d29e https://code.google.com/p/go.talks #

Patch Set 5 : diff -r 44382830d29e https://code.google.com/p/go.talks #

Patch Set 6 : diff -r 44382830d29e https://code.google.com/p/go.talks #

Patch Set 7 : diff -r 44382830d29e https://code.google.com/p/go.talks #

Patch Set 8 : diff -r 44382830d29e https://code.google.com/p/go.talks #

Patch Set 9 : diff -r 4dc5255056df https://code.google.com/p/go.talks #

Patch Set 10 : diff -r 4dc5255056df https://code.google.com/p/go.talks #

Patch Set 11 : diff -r 4dc5255056df https://code.google.com/p/go.talks #

Patch Set 12 : diff -r 4dc5255056df https://code.google.com/p/go.talks #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+1248 lines, -0 lines) Patch
A 2012/goforc.slide View 1 2 3 4 5 6 7 8 9 10 1 chunk +726 lines, -0 lines 1 comment Download
A 2012/goforc/adder.go View 1 2 1 chunk +21 lines, -0 lines 0 comments Download
A 2012/goforc/cat.go View 1 2 3 1 chunk +22 lines, -0 lines 0 comments Download
A 2012/goforc/celsius.go View 1 1 chunk +17 lines, -0 lines 0 comments Download
A 2012/goforc/channels.go View 1 1 chunk +26 lines, -0 lines 0 comments Download
A 2012/goforc/communication1.go View 1 1 chunk +28 lines, -0 lines 0 comments Download
A 2012/goforc/communication2.go View 1 1 chunk +30 lines, -0 lines 0 comments Download
A 2012/goforc/consts.go View 1 2 1 chunk +24 lines, -0 lines 0 comments Download
A 2012/goforc/decls.go View 1 1 chunk +23 lines, -0 lines 0 comments Download
A 2012/goforc/example0.go View 1 1 chunk +23 lines, -0 lines 0 comments Download
A 2012/goforc/example1.go View 1 1 chunk +22 lines, -0 lines 0 comments Download
A 2012/goforc/example2.go View 1 1 chunk +24 lines, -0 lines 0 comments Download
A 2012/goforc/forloop.go View 1 2 1 chunk +28 lines, -0 lines 0 comments Download
A 2012/goforc/hello.go View 1 1 chunk +7 lines, -0 lines 0 comments Download
A 2012/goforc/interface.go View 1 1 chunk +34 lines, -0 lines 0 comments Download
A 2012/goforc/point.go View 1 1 chunk +20 lines, -0 lines 0 comments Download
A 2012/goforc/stmts.go View 1 1 chunk +33 lines, -0 lines 0 comments Download
A 2012/goforc/vars.go View 1 2 3 4 5 1 chunk +18 lines, -0 lines 0 comments Download
A 2012/goforc/worker1.go View 1 2 3 4 5 6 7 8 1 chunk +60 lines, -0 lines 0 comments Download
A 2012/goforc/worker2.go View 1 2 3 4 5 6 7 8 9 10 1 chunk +62 lines, -0 lines 0 comments Download

Messages

Total messages: 6
gri
Hello r@golang.org (cc: golang-dev@googlegroups.com), I'd like you to review this change to https://code.google.com/p/go.talks
11 years, 5 months ago (2012-11-16 01:45:39 UTC) #1
r
LGTM
11 years, 5 months ago (2012-11-16 19:11:03 UTC) #2
gri
*** Submitted as http://code.google.com/p/go/source/detail?r=6f52cd2d3710&repo=talks *** 2012/goforc: Slides for "Go for C programmers" talk. R=r CC=golang-dev ...
11 years, 5 months ago (2012-11-16 19:53:19 UTC) #3
minux1
http://codereview.appspot.com/6782087 https://codereview.appspot.com/6848048/diff/11021/2012/goforc.slide File 2012/goforc.slide (right): https://codereview.appspot.com/6848048/diff/11021/2012/goforc.slide#newcode388 2012/goforc.slide:388: Support for multiple platforms: x86 (32/64bit), ARM (32/64), ...
11 years, 5 months ago (2012-11-18 20:26:28 UTC) #4
gri
That's a copy/paste error I think. But gcc has just gotten support for ARM 64, ...
11 years, 5 months ago (2012-11-18 21:18:13 UTC) #5
iant2
11 years, 5 months ago (2012-11-19 03:03:44 UTC) #6
On Sun, Nov 18, 2012 at 1:18 PM, Robert Griesemer <gri@golang.org> wrote:
> That's a copy/paste error I think. But gcc has just gotten support for
> ARM 64, I believe. Ian?

Yes, GCC 4.8, including gccgo, will support Aarch64 (ARM does not want
the processors to be called ARM32 and ARM64, although that does seem
to be the names that people tend to use).

Ian
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b