[Freeciv-Dev] Re: clistat

[David Pfitzner <dwp@xxxxxxxxxxxxxx>]

Vasco <vasc@xxxxxxxxxxxxxx> wrote:

> On second thought...  The send_buffer should be dynamic (i think David was
> planning this recently).

Well, I wanted it dynamically allocated for a different reason:
to not waste space for unused connection structs, and for
"partly-used" structs in the client.  But feel free to use it now 
for re-allocing if the buffer gets too full!  (It _is_ now malloced 
in CVS version.)

>  Whenever it goes full, it should grow in
> MAX_LEN_PACKET steps.  It could start with the current size of
> 10*MAX_LEN_PACKET.  This should make "lost" packets a rarer event (unless
> the send_buffer gets REALLY big (like 1Mb), or realloc fails - in that
> case we probably should cut that player connection.

Sure.

-- David