xpra icon
Bug tracker and wiki

Opened 3 months ago

Last modified 5 weeks ago

#1942 assigned enhancement

new packet format: dictionary instead of positional arguments

Reported by: Antoine Martin Owned by: Antoine Martin
Priority: major Milestone: 2.5
Component: core Version: 2.3.x
Keywords: Cc:

Description

We can support both the old format and the new one for a while.
The new one will have the advantage of being easier to understand and more flexible so we can add new features, make some parts optional, without having to fill in positional arguments we may not have. (they may not be relevant or may not be useful)

Change History (2)

comment:1 Changed 3 months ago by Antoine Martin

Status: newassigned
Summary: new configure packet format: dictionary instead of positional argumentsnew packet format: dictionary instead of positional arguments

Example of ugly packet formats which would benefit from this: r20252 (adding relative position to the existing position parameter to avoid adding it at the end of the packet, which has optional fields..), configure packets, map packets, etc...

We can continue to support the old format, and convert to the new one - and maybe the same conversion function could be used by both the client and server to keep the code simple.

comment:2 Changed 8 weeks ago by Antoine Martin

Milestone: 3.02.5

Could also be used for keyboard (#1172) and refresh (#1449).

On a more general note: info packets and #1983, moving keyboard packets out of hello, hello namespace (#1154) are all related to legacy formats.

Last edited 5 weeks ago by Antoine Martin (previous) (diff)
Note: See TracTickets for help on using tickets.