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

Issue 6585074: code review 6585074: golang-tour: Solutions to the exercises. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years, 5 months ago by francesc
Modified:
12 years, 5 months ago
Reviewers:
CC:
adg, golang-dev
Visibility:
Public.

Description

golang-tour: Solutions to the exercises.

Patch Set 1 #

Patch Set 2 : diff -r af614bf4623b https://code.google.com/p/go-tour #

Patch Set 3 : diff -r af614bf4623b https://code.google.com/p/go-tour #

Total comments: 1

Patch Set 4 : diff -r 4914aa37f57f https://code.google.com/p/go-tour #

Unified diffs Side-by-side diffs Delta from patch set Stats (+428 lines, -0 lines) Patch
A gotour/solutions/README View 1 1 chunk +8 lines, -0 lines 0 comments Download
A gotour/solutions/binarytrees.go View 1 1 chunk +61 lines, -0 lines 0 comments Download
A gotour/solutions/complexcube.go View 1 1 chunk +26 lines, -0 lines 0 comments Download
A gotour/solutions/errors.go View 1 1 chunk +34 lines, -0 lines 0 comments Download
A gotour/solutions/fib.go View 1 1 chunk +20 lines, -0 lines 0 comments Download
A gotour/solutions/image.go View 1 1 chunk +43 lines, -0 lines 0 comments Download
A gotour/solutions/loops.go View 1 1 chunk +26 lines, -0 lines 0 comments Download
A gotour/solutions/maps.go View 1 1 chunk +18 lines, -0 lines 0 comments Download
A gotour/solutions/rot13.go View 1 1 chunk +39 lines, -0 lines 0 comments Download
A gotour/solutions/slices.go View 1 1 chunk +22 lines, -0 lines 0 comments Download
A gotour/solutions/webcrawler.go View 1 1 chunk +131 lines, -0 lines 0 comments Download

Messages

Total messages: 3
francesc
Hello adg (cc: golang-dev@googlegroups.com), I'd like you to review this change to https://code.google.com/p/go-tour
12 years, 5 months ago (2012-10-03 23:02:50 UTC) #1
adg
LGTM http://codereview.appspot.com/6585074/diff/5001/gotour/solutions/binarytrees.go File gotour/solutions/binarytrees.go (right): http://codereview.appspot.com/6585074/diff/5001/gotour/solutions/binarytrees.go#newcode1 gotour/solutions/binarytrees.go:1: package main I guess you should add the ...
12 years, 5 months ago (2012-10-03 23:14:38 UTC) #2
francesc
12 years, 5 months ago (2012-10-03 23:24:23 UTC) #3
*** Submitted as http://code.google.com/p/go-tour/source/detail?r=bb8e3b14b929
***

golang-tour: Solutions to the exercises.

R=adg
CC=golang-dev
http://codereview.appspot.com/6585074
Sign in to reply to this message.

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