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

Issue 12082045: code review 12082045: net: add missing SetKeepAlivePeriod for Plan 9 (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years, 11 months ago by mikio
Modified:
11 years, 11 months ago
Reviewers:
dave, golang-dev
CC:
golang-dev
Visibility:
Public.

Description

net: add missing SetKeepAlivePeriod for Plan 9

Patch Set 1 #

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

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

Patch Set 4 : diff -r 7c42cfa28e24 https://code.google.com/p/go #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -0 lines) Patch
M src/pkg/net/tcpsock_plan9.go View 1 1 chunk +5 lines, -0 lines 0 comments Download

Messages

Total messages: 3
mikio
Hello golang-dev@googlegroups.com (cc: golang-dev@googlegroups.com), I'd like you to review this change to https://code.google.com/p/go
11 years, 11 months ago (2013-07-30 05:50:01 UTC) #1
dave_cheney.net
LGTM. Thanks On Tue, Jul 30, 2013 at 3:50 PM, <mikioh.mikioh@gmail.com> wrote: > Reviewers: golang-dev1, ...
11 years, 11 months ago (2013-07-30 06:21:46 UTC) #2
mikio
11 years, 11 months ago (2013-07-30 07:33:49 UTC) #3
For the record; just submitted but got "ssl.SSLError: The read
operation timed out" during the submission process. Looks like build
dashboard is fine.
Sign in to reply to this message.

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