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

Issue 1199045: code review 1199045: Add support for Socketpair

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

Description

Add support for Socketpair

Patch Set 1 #

Patch Set 2 : code review 1199045: Add support for Socketpair #

Patch Set 3 : code review 1199045: Add support for Socketpair #

Unified diffs Side-by-side diffs Delta from patch set Stats (+55 lines, -3 lines) Patch
M src/pkg/syscall/syscall_bsd.go View 1 1 chunk +6 lines, -0 lines 0 comments Download
M src/pkg/syscall/syscall_linux.go View 1 1 chunk +5 lines, -1 line 0 comments Download
M src/pkg/syscall/syscall_linux_386.go View 1 chunk +4 lines, -0 lines 0 comments Download
M src/pkg/syscall/syscall_linux_amd64.go View 1 chunk +1 line, -0 lines 0 comments Download
M src/pkg/syscall/syscall_linux_arm.go View 1 chunk +1 line, -0 lines 0 comments Download
M src/pkg/syscall/zsyscall_darwin_386.go View 1 chunk +6 lines, -0 lines 0 comments Download
M src/pkg/syscall/zsyscall_darwin_amd64.go View 1 chunk +6 lines, -0 lines 0 comments Download
M src/pkg/syscall/zsyscall_freebsd_386.go View 2 chunks +7 lines, -1 line 0 comments Download
M src/pkg/syscall/zsyscall_freebsd_amd64.go View 2 chunks +7 lines, -1 line 0 comments Download
M src/pkg/syscall/zsyscall_linux_amd64.go View 1 chunk +6 lines, -0 lines 0 comments Download
M src/pkg/syscall/zsyscall_linux_arm.go View 1 chunk +6 lines, -0 lines 0 comments Download

Messages

Total messages: 3
taruti
Hello golang-dev@googlegroups.com, I'd like you to review this change.
15 years, 5 months ago (2010-05-15 17:49:49 UTC) #1
taruti
Hello golang-dev@googlegroups.com (cc: golang-dev@googlegroups.com), Please take another look.
15 years, 5 months ago (2010-05-21 07:29:54 UTC) #2
rsc1
15 years, 5 months ago (2010-06-03 01:01:40 UTC) #3
There's another CL trying to add Socketpair too. I'm not sure which was here first, but since I looked at the other one more recently, I'm going to leave this one alone. Sorry for the duplicated effort.
Sign in to reply to this message.

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