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

Issue 5532088: code review 5532088: time: add Since, which returns the time elapsed since s... (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
13 years, 3 months ago by Sameer Ajmani
Modified:
13 years, 3 months ago
Reviewers:
CC:
rsc, r, golang-dev
Visibility:
Public.

Description

time: add Since, which returns the time elapsed since some past time t.

Patch Set 1 #

Patch Set 2 : diff -r 9f2cc72b8e0d https://go.googlecode.com/hg/ #

Patch Set 3 : diff -r 9f2cc72b8e0d https://go.googlecode.com/hg/ #

Patch Set 4 : diff -r f1e930b56e11 https://go.googlecode.com/hg/ #

Unified diffs Side-by-side diffs Delta from patch set Stats (+19 lines, -13 lines) Patch
M src/pkg/time/time.go View 1 7 chunks +19 lines, -13 lines 0 comments Download

Messages

Total messages: 7
Sameer Ajmani
Hello golang-dev@googlegroups.com, I'd like you to review this change to https://go.googlecode.com/hg/
13 years, 3 months ago (2012-01-18 15:54:30 UTC) #1
rsc
Looks fine. Was there anything else we had discussed? I can't find the email thread.
13 years, 3 months ago (2012-01-18 18:52:46 UTC) #2
r
LGTM although I don't encourage a new method/function for every arithmetic operation.
13 years, 3 months ago (2012-01-18 19:15:35 UTC) #3
Sameer Ajmani
On 2012/01/18 18:52:46, rsc wrote: > Looks fine. Was there anything else we had discussed? ...
13 years, 3 months ago (2012-01-18 19:51:28 UTC) #4
Sameer Ajmani
On 2012/01/18 19:15:35, r wrote: > LGTM although I don't encourage a new method/function for ...
13 years, 3 months ago (2012-01-18 19:52:04 UTC) #5
rsc
LGTM Since makes Elapsed not interesting. UnixMilli and UnixMicro don't seem so relevant here, so ...
13 years, 3 months ago (2012-01-18 20:02:35 UTC) #6
Sameer Ajmani
13 years, 3 months ago (2012-01-18 20:25:32 UTC) #7
*** Submitted as http://code.google.com/p/go/source/detail?r=a56668df61e3 ***

time: add Since, which returns the time elapsed since some past time t.

R=rsc, r
CC=golang-dev
http://codereview.appspot.com/5532088
Sign in to reply to this message.

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