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

Issue 875041: code review 875041: runtime: correct memory leak in select (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
14 years, 12 months ago by rsc
Modified:
14 years, 11 months ago
Reviewers:
ken3
CC:
ken2, golang-dev, r
Visibility:
Public.

Description

runtime: correct memory leak in select * adds pass 3 to dequeue from channels eagerly various other cleanup/churn: * use switch on cas->send in each pass to factor out common code. * longer goto labels, commented at target * be more agressive about can't happen: throw instead of print + cope. * use "select" instead of "selectgo" in errors * use printf for debug prints when possible

Patch Set 1 #

Patch Set 2 : code review 875041: runtime: correct memory leak in select #

Patch Set 3 : code review 875041: runtime: correct memory leak in select #

Patch Set 4 : code review 875041: runtime: correct memory leak in select #

Unified diffs Side-by-side diffs Delta from patch set Stats (+180 lines, -186 lines) Patch
M src/pkg/runtime/chan.c View 1 2 3 21 chunks +132 lines, -186 lines 0 comments Download
A test/chan/select2.go View 1 chunk +48 lines, -0 lines 0 comments Download

Messages

Total messages: 4
rsc
Hello ken2 (cc: golang-dev@googlegroups.com, r), I'd like you to review this change.
14 years, 12 months ago (2010-04-01 08:11:06 UTC) #1
ken3
LGTM On 2010/04/01 08:11:06, rsc wrote: > Hello ken2 (cc: mailto:golang-dev@googlegroups.com, r), > > I'd ...
14 years, 12 months ago (2010-04-01 18:51:03 UTC) #2
rsc
*** Submitted as http://code.google.com/p/go/source/detail?r=81ccd24eb407 *** runtime: correct memory leak in select * adds pass 3 ...
14 years, 12 months ago (2010-04-01 18:56:22 UTC) #3
ken3
14 years, 11 months ago (2010-04-05 23:13:29 UTC) #4
LGTM
Sign in to reply to this message.

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