Opened 6 years ago

Closed 3 years ago

#1046 closed defect (fixed)

Streaming: Optional delay always set

Reported by: zzz Owned by:
Priority: minor Milestone: 0.9.29
Component: streaming Version: 0.9.7.1
Keywords: docs cleanup Cc: Zlatin Balevsky
Parent Tickets: Sensitive: no

Description

Even when 0. See Connection.sendPacket(). Two bytes of overhead per-packet. We should be able to remove it if zero, but check handling on receive side.

Subtickets

Change History (7)

comment:1 Changed 6 years ago by Zlatin Balevsky

Cc: Zlatin Balevsky added

comment:2 Changed 6 years ago by zzz

Milestone: 0.9.90.9.15

An optional delay with a value of zero means 'ack immediately' - see ConnectionPacketHandler?.receivePacket().

Need a more thorough review of the docs and code to see if what we're doing is sensible and consistent.

comment:3 Changed 5 years ago by str4d

Keywords: docs cleanup added
Milestone: 0.9.15

comment:4 Changed 4 years ago by str4d

Status: newopen

comment:5 Changed 3 years ago by zzz

Milestone: 0.9.29
Resolution: fixed
Status: openclosed

comment:6 Changed 3 years ago by zzz

Resolution: fixed
Status: closedreopened

still set in multiple places. will be resolved in #1939

comment:7 Changed 3 years ago by zzz

Resolution: fixed
Status: reopenedclosed
Note: See TracTickets for help on using tickets.