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

Issue 91090043: code review 91090043: cmd/ld: remove vestigial subroutines (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
9 years, 11 months ago by ality
Modified:
9 years, 7 months ago
Visibility:
Public.

Description

cmd/ld: remove vestigial subroutines Worn out and weary Remnants of the exodus Live on in liblink

Patch Set 1 #

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

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

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -109 lines) Patch
M src/cmd/ld/lib.h View 1 2 4 chunks +0 lines, -5 lines 0 comments Download
R src/cmd/ld/pass.c View 1 1 chunk +0 lines, -104 lines 0 comments Download

Messages

Total messages: 10
ality
Hello golang-codereviews@googlegroups.com, I'd like you to review this change to https://code.google.com/p/go
9 years, 11 months ago (2014-05-05 06:31:31 UTC) #1
bradfitz
LGTM Syllables check out. Lacks seasonal reference but will accept it. On May 5, 2014 ...
9 years, 11 months ago (2014-05-05 11:56:01 UTC) #2
josharian
> > Worn out and weary > > Remnants of the exodus > > Live ...
9 years, 11 months ago (2014-05-05 13:45:15 UTC) #3
bradfitz
On Mon, May 5, 2014 at 9:45 AM, <josharian@gmail.com> wrote: > > Worn out and ...
9 years, 11 months ago (2014-05-05 14:08:00 UTC) #4
crawshaw
On Mon, May 5, 2014 at 10:07 AM, Brad Fitzpatrick <bradfitz@golang.org>wrote: > On Mon, May ...
9 years, 11 months ago (2014-05-05 15:32:37 UTC) #5
josharian
>>>> > Worn out and weary >>>> > Remnants of the exodus >>>> > Live ...
9 years, 11 months ago (2014-05-05 16:02:58 UTC) #6
adg
On 2014/05/05 16:02:58, josharian wrote: > >>>> > Worn out and weary > >>>> > ...
9 years, 11 months ago (2014-05-06 18:51:05 UTC) #7
bradfitz
‘Twas gophy, and the slithy codes, did go and defer from their Ms, All blocky ...
9 years, 11 months ago (2014-05-06 20:53:53 UTC) #8
dave_cheney.net
ping. ality, are you going to submit this ?
9 years, 9 months ago (2014-07-16 10:52:45 UTC) #9
ality
9 years, 7 months ago (2014-09-09 21:13:06 UTC) #10
On 2014/07/16 10:52:45, dfc wrote:
> ping. ality, are you going to submit this ?

This was accomplished by CL 120220043 (changeset e1156a6baee2).

Closing.
Sign in to reply to this message.

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