search

astrand / xclip

Command line interface to the X11 clipboard
GNU General Public License v2.0
1.04k stars 75 forks source link

The -loops option is broken after merging the pull-request #57 #59

Closed hwangcc23 closed 5 years ago

hwangcc23 commented 5 years ago

The -loops option is broken after merging the pull-request #57.

Reported by @mckellyln.

hwangcc23 commented 5 years ago

Since the pull-request #57 changed the code for large-data-transfers, I tried to transfer large data and force ONLY one loop ("-loop 1" option).

Then I reproduced the problem using this command: hwangcc23-BU401LG:~$ dd 'if=/dev/zero' 'bs=2000' 'count=1000' | tr '\000' 'a' | xclip -i -quiet -l 1

Expect: xclip should exist right after any requestor retrieved data. What I saw: xclip didn't exist.

I will send a PR to fix this problem.

hwangcc23 commented 5 years ago

@mckellyln I have sent a PR to fix the problem. Could you try out the patch?

mckellyln commented 5 years ago

@hwangcc23, @astrand I think we can close this now.