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

Issue 68210044: Add note to flow control about ENOBUFS for UDP on OSX/BSD (Closed)

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

Description

Add note to flow control about ENOBUFS for UDP on OSX/BSD

Patch Set 1 #

Patch Set 2 : Slight text improvements #

Total comments: 2

Patch Set 3 : Use markup to link ENOBUFS to its docs #

Unified diffs Side-by-side diffs Delta from patch set Stats (+11 lines, -2 lines) Patch
M Doc/library/asyncio-protocol.rst View 1 2 2 chunks +11 lines, -2 lines 0 comments Download

Messages

Total messages: 6
GvR
11 years, 11 months ago (2014-02-24 19:47:57 UTC) #1
yselivanov
https://codereview.appspot.com/68210044/diff/20001/Doc/library/asyncio-protocol.rst File Doc/library/asyncio-protocol.rst (right): https://codereview.appspot.com/68210044/diff/20001/Doc/library/asyncio-protocol.rst#newcode410 Doc/library/asyncio-protocol.rst:410: :class:`OSError` with errno set to ENOBUFS may or may ...
11 years, 11 months ago (2014-02-24 20:04:16 UTC) #2
GvR
Use markup to link ENOBUFS to its docs
11 years, 11 months ago (2014-02-24 20:10:04 UTC) #3
GvR
https://codereview.appspot.com/68210044/diff/20001/Doc/library/asyncio-protocol.rst File Doc/library/asyncio-protocol.rst (right): https://codereview.appspot.com/68210044/diff/20001/Doc/library/asyncio-protocol.rst#newcode410 Doc/library/asyncio-protocol.rst:410: :class:`OSError` with errno set to ENOBUFS may or may ...
11 years, 11 months ago (2014-02-24 20:10:52 UTC) #4
gvrpython
Can I have an LGTM? On Feb 24, 2014 12:10 PM, <gvanrossum@gmail.com> wrote: > > ...
11 years, 11 months ago (2014-02-24 22:25:22 UTC) #5
yselivanov
11 years, 11 months ago (2014-02-24 22:26:15 UTC) #6
LGTM from me.
Well written and easy to understand.
Sign in to reply to this message.

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