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

Issue 108750043: Fill more XXX docstrings

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

Description

Fill more XXX docstrings

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+53 lines, -7 lines) Patch
M asyncio/base_events.py View 3 chunks +16 lines, -3 lines 2 comments Download
M asyncio/selector_events.py View 4 chunks +37 lines, -4 lines 0 comments Download

Messages

Total messages: 1
GvR
9 years, 10 months ago (2014-06-05 14:52:32 UTC) #1
(Rest looks good.)

https://codereview.appspot.com/108750043/diff/1/asyncio/base_events.py
File asyncio/base_events.py (right):

https://codereview.appspot.com/108750043/diff/1/asyncio/base_events.py#newcod...
asyncio/base_events.py:361: """Create a streaming transport connection to a
given Internet host and
Argh! I want docstrings to have one short description that fits on the first
line, then a blank line, then more stuff as needed.

https://codereview.appspot.com/108750043/diff/1/asyncio/base_events.py#newcod...
asyncio/base_events.py:569: port.
Be creative. Make it fit.
Sign in to reply to this message.

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