This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author gvanrossum
Recipients gvanrossum, martin.panter, vstinner, yselivanov
Date 2014-12-17.00:39:30
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <CAP7+vJL-Yp6vQtUaQ0F46dXC--G=x7+P+Dw9MesPRqtohG6L=w@mail.gmail.com>
In-reply-to <CAMpsgwa_KtSAVD6F1F7821NQG8V1s9EvpUKwO_QLmKrOiQg=9Q@mail.gmail.com>
Content
Heh. Well I don't remember why I did that any more, and it doesn't seem to
matter now.

However the doc issue seems different than for BaseEventLoop -- Server is
the *concrete* class (it actually gets instantiated, not a subclass). We
could instead document the AbstractServer class, but it doesn't have the
'sockets' instance variable. Maybe we should document both --
AbstractServer as the minimal interface that create_server() returns,
Server as the actual class that the default event loops (Unix and Windows)
create. With a warning that 'sockets' attribute may not be available if the
event loop has been configured differently.

On Tue, Dec 16, 2014 at 4:27 PM, STINNER Victor <report@bugs.python.org>
wrote:
>
>
> STINNER Victor added the comment:
>
> > why was Server there at all? Tests?
>
> If you cannot answer, who can answer? :-)
> https://code.google.com/p/tulip/source/detail?r=f136c04d82c0 (You
> added Server to __all__.)
>
> I don't see any use case which needs to create explicitly a Server
> class. There are the create_server() method and start_server()
> function for that.
>
> By the way, the Server class *is* documented as asyncio.Server, which
> is the same mistake than asyncio.BaseEventLoop:
> https://docs.python.org/dev/library/asyncio-eventloop.html#asyncio.Server
>
> I propose to update the doc for Server, replace asyncio.Server with
> asyncio.base_events.Server.
>
> ----------
>
> _______________________________________
> Python tracker <report@bugs.python.org>
> <http://bugs.python.org/issue23046>
> _______________________________________
>
History
Date User Action Args
2014-12-17 00:39:30gvanrossumsetrecipients: + gvanrossum, vstinner, martin.panter, yselivanov
2014-12-17 00:39:30gvanrossumlinkissue23046 messages
2014-12-17 00:39:30gvanrossumcreate