Rémi Bernon (@rbernon) commented about server/protocol.def:
int cursor_count; /* cursor show count */ unsigned char keystate[256]; /* key state */ int keystate_lock; /* keystate is locked */
- unsigned __int64 desktop_keystate_serial; /* desktop keystate update counter at last sync */
I don't see a reason to prefix it with "desktop_", lets name it like the keystate field which is mirrored from desktop shm.
```suggestion:-2+0 unsigned char keystate[256]; /* key state */ unsigned __int64 keystate_serial; /* keystate update counter */ int keystate_lock; /* keystate is locked */ ```