Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

speed up clipboard paste for "greedy" clients (win32) #313

Closed
totaam opened this issue Apr 16, 2013 · 1 comment
Closed

speed up clipboard paste for "greedy" clients (win32) #313

totaam opened this issue Apr 16, 2013 · 1 comment
Labels
clipboard win32 MS Windows OS

Comments

@totaam
Copy link
Collaborator

totaam commented Apr 16, 2013

Issue migrated from trac ticket # 313

component: client | priority: minor | resolution: wontfix

2013-04-16 07:29:33: totaam created the issue


Not sure if this would work or not, OTOH why not?
[[BR]]
When we paste something coming from the server into a win32 application, we ask for the clipboard contents everytime the client application requests the data (for every paste).
[[BR]]
Because this ends up doing a round-trip to the server, this can be quite slow. For example: try copying something in the server, then holding "Ctrl+V" in notepad on win32.
[[BR]]
Because win32 clients are "greedy" / buggy, we are already notified by the server whenever the contents change so we should be able to cache the value after each request (per TARGET: UTF8_STRING="somestring", STRING="..") and skip the roundtrip for subsequent requests.
[[BR]]
Furthermore, if this works well enough, we may want to make all clients greedy to speed things up? (or maybe just for "CLIPBOARD" as "PRIMARY" changes way too often - beware of clipboard managers problems though)


Another thing we may be able to do for greedy clients is to always send the list of targets with the clipboard token, saving another round trip as the client requests the targets pretty much every time (IIRC).

@totaam
Copy link
Collaborator Author

totaam commented Jul 11, 2013

2013-07-11 17:53:14: antoine commented


Clipboard is tricky enough as it is!

@totaam totaam closed this as completed Jul 11, 2013
@totaam totaam added clipboard win32 MS Windows OS labels Jan 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
clipboard win32 MS Windows OS
Projects
None yet
Development

No branches or pull requests

1 participant