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

What is the use case for -loops? #105

Open
hackerb9 opened this issue Nov 6, 2020 · 0 comments
Open

What is the use case for -loops? #105

hackerb9 opened this issue Nov 6, 2020 · 0 comments

Comments

@hackerb9
Copy link
Collaborator

hackerb9 commented Nov 6, 2020

What is the purpose of -loops? In trying to figure out how to fix -loops in bug #73, I'm coming to the conclusion that they will be complex and bring subtle errors in the future. Are they worth it?

Here is the comment I made in that bug report about what we can do to fix -loops for right now and questions we'll need to address in the future:


Certain programs send many more paste requests than they should. For example, middle-click pasting into this very message using Mozilla Firefox triggered 21 paste requests, the first sixteen of which are simply duplicate requests for TARGETS (the list of available targets). Since requests for special targets like TARGETS, TIMESTAMP, and MULTIPLE aren't genuine pastes, they should not increment dloops. This should be fairly easy for us to fix.

However, that still leaves five paste requests for that single middle click. The browser asks for five different targets: text/plain;charset=utf-8, UTF8_STRING, COMPOUND_TEXT, STRING, and text/x-moz-text-internal. Since xclip only holds one format, xclip should decline the other four of those requests and not increment dloops. (This is bug #94).

Once those two problems are solved, we should reevaluate whether the loops feature is still broken.

Originally posted by @hackerb9 in #73 (comment)

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

No branches or pull requests

1 participant