Skip to content

Conversation

@ns6089
Copy link
Contributor

@ns6089 ns6089 commented Jan 3, 2026

Implements #120 proposal.

Notes:

  • Code is cleaned up and ready for review.
  • Tested on this simple sunshine mock commit and moonlight-qt. LTR ACK messages are being received by sunshine and I didn't break RFI.
  • From the performance standpoint it probably makes sense to batch LTR ACK messages together with input so we won't use extra radio time when on wireless. But I didn't trust myself to be able to do this in a non-ugly way. If somebody gets to that in the future, per contract server will be able to handle LTR ACK messages that arrive late and out of order in relation to RFI requests.
  • Since it doesn't affect existing functionality and given the release cycle of moonlight-qt, it would be extremely nice to incorporate this into 6.2 release 🙏

@ns6089 ns6089 force-pushed the ltr_ack branch 3 times, most recently from 275d274 to 3f91ab3 Compare January 4, 2026 20:44
@ns6089 ns6089 marked this pull request as ready for review January 5, 2026 08:49
@cgutman
Copy link
Member

cgutman commented Jan 21, 2026

There's a missing initializer warning that's causing CI to fail, but otherwise LGTM.

@cgutman cgutman merged commit 2a5a1f3 into moonlight-stream:master Jan 21, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants