Opened 10 years ago
Closed 9 years ago
#5918 enhancement closed fixed (fixed)
t.p.b.StatefulStringProtocol has undocumented .state instance attribute
Reported by: | Devin Jeanpierre | Owned by: | Thijs Triemstra |
---|---|---|---|
Priority: | normal | Milestone: | |
Component: | core | Keywords: | documentation |
Cc: | Thijs Triemstra | Branch: |
branches/state-doc-5918
branch-diff, diff-cov, branch-cov, buildbot |
Author: | thijs |
Description
http://twistedmatrix.com/trac/browser/tags/releases/twisted-12.1.0/twisted/protocols/basic.py#L846
http://twistedmatrix.com/documents/current/api/twisted.protocols.basic.StatefulStringProtocol.html
I'm not sure that instance attributes are universally supposed to be documented, but the attribute name "state" is useful enough for others (and nonobvious enough in subclasses, e.g. AMP) that maybe it should be documented. I ran into this problem last year when I used state as an instance attribute for a subclass, and found it very confusing. (At the time, I forgot to report the missing documentation, sorry.)
Change History (6)
comment:1 Changed 10 years ago by
Cc: | Thijs Triemstra added |
---|---|
Keywords: | documentation added |
comment:2 Changed 9 years ago by
Author: | → thijs |
---|---|
Branch: | → branches/state-doc-5918 |
comment:4 Changed 9 years ago by
Keywords: | review added |
---|
Added documentation and forced a build. Up for review.
comment:5 Changed 9 years ago by
Keywords: | review removed |
---|---|
Owner: | set to Thijs Triemstra |
Looks good. Please commit.
comment:6 Changed 9 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
(In [37212]) Branching to 'state-doc-5918'