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

Issue 128310043: code review 128310043: cmd/ld: fix operator precedence (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 8 months ago by dave
Modified:
10 years, 8 months ago
Reviewers:
gobot, minux
CC:
minux, golang-codereviews
Visibility:
Public.

Description

cmd/ld: fix operator precedence Fixes issue 8480. This CL reapplies CL 114420043. This attempt doesn't blow up when encountering hidden symbols.

Patch Set 1 #

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

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

Patch Set 4 : code review 128310043: cmd/ld: fix operator precedence #

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

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

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M src/cmd/ld/data.c View 1 2 3 4 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 7
dave_cheney.net
Hello minux@golang.org (cc: golang-codereviews@googlegroups.com), I'd like you to review this change to https://code.google.com/p/go
10 years, 8 months ago (2014-08-15 12:22:45 UTC) #1
minux
LGTM.
10 years, 8 months ago (2014-08-16 03:52:07 UTC) #2
dave_cheney.net
Submitting, here goes nothing. On Sat, Aug 16, 2014 at 1:52 PM, <minux@golang.org> wrote: > ...
10 years, 8 months ago (2014-08-16 04:04:02 UTC) #3
dave_cheney.net
*** Submitted as https://code.google.com/p/go/source/detail?r=026598be012a *** cmd/ld: fix operator precedence Fixes issue 8480. This CL reapplies ...
10 years, 8 months ago (2014-08-16 04:10:50 UTC) #4
gobot
This CL appears to have broken the netbsd-386-minux builder. See http://build.golang.org/log/c0918aed39fd6bc2cb4a63c248263bb1a0bd7fee
10 years, 8 months ago (2014-08-16 04:28:36 UTC) #5
dave_cheney.net
Unrelated, ok net/url 0.003s --- FAIL: TestKillFindProcess (4.08s) os_test.go:1319: Test process succeeded, but expected to ...
10 years, 8 months ago (2014-08-16 04:37:12 UTC) #6
minux
10 years, 8 months ago (2014-08-16 04:39:34 UTC) #7
On Sat, Aug 16, 2014 at 12:37 AM, Dave Cheney <dave@cheney.net> wrote:

> Unrelated,
>
> ok   net/url 0.003s
> --- FAIL: TestKillFindProcess (4.08s)
> os_test.go:1319: Test process succeeded, but expected to fail
> FAIL
> FAIL os 4.467s

I used to believe such failures are due to bad interaction with tmux,
but the builder has been running inside GNU screen for a long time
now. I don't understand why it still happens occasionally, and it's not
very reproducible.
Sign in to reply to this message.

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