Path: csiph.com!v102.xanadu-bbs.net!xanadu-bbs.net!feeder.erje.net!eu.feeder.erje.net!news.stack.nl!newsfeed.xs4all.nl!newsfeed4.news.xs4all.nl!xs4all!newsgate.cistron.nl!newsgate.news.xs4all.nl!post.news.xs4all.nl!not-for-mail
To: python-list@python.org
From: "Frank Millman" <frank@chagford.com>
Subject: Re: Learning python networking
Date: Wed, 15 Jan 2014 15:31:15 +0200
References: <22d58d76-f2c2-4a1d-8049-3409ac4665d3@googlegroups.com> <mailman.5207.1389223748.18130.python-list@python.org> <9202d352-e065-4f2b-a9e0-e29ce5c68df6@googlegroups.com> <CAPTjJmpb6yr-VpWypbJQn0a=pNjvNV2CchVBZaK+v_5JoSQOBg@mail.gmail.com>
Precedence: list
Newsgroups: comp.lang.python
Message-ID: <mailman.5512.1389792688.18130.python-list@python.org>
Lines: 42
NNTP-Posting-Host: 2001:888:2000:d::a6
Xref: csiph.com comp.lang.python:63985


"Chris Angelico" <rosuav@gmail.com> wrote in message 
news:CAPTjJmpb6yr-VpWypbJQn0a=pNjvNV2CchVBZaK+v_5JoSQOBg@mail.gmail.com...
> You just run a loop like this:
>
> buffer = b''
>
> def gets():
>    while '\n' not in buffer:
>        data = sock.recv(1024)
>        if not data:
>            # Client is disconnected, handle it gracefully
>            return None # or some other sentinel
>    line, buffer = buffer.split(b'\n',1)
>    return line.decode().replace('\r', '')
>

I think you may have omitted a line there -

def gets():
    while '\n' not in buffer:
        data = sock.recv(1024)
        if not data:
            # Client is disconnected, handle it gracefully
            return None # or some other sentinel
#-->
        buffer = buffer + data
#-->
    line, buffer = buffer.split(b'\n',1)
    return line.decode().replace('\r', '')

Also, as I am looking at it, I notice that the second line should say -

    while b'\n' not in buffer:

I feel a bit guilty nitpicking, as you have provided a wonderfully 
comprehensive answer, but I wanted to make sure the OP did not get confused.

Frank Millman