[hackers] [st] delete clipboard properties after pasting them || v4hn

From: <git_AT_suckless.org>
Date: Fri, 3 Jun 2016 15:03:51 +0200 (CEST)

commit 60aeb37edb8c5280d31b6b3c801d09c7a5fdca76
Author: v4hn <me_AT_v4hn.de>
AuthorDate: Thu May 19 12:16:57 2016 +0200
Commit: Christoph Lohmann <20h_AT_r-36.net>
CommitDate: Fri Jun 3 14:55:25 2016 +0200

    delete clipboard properties after pasting them
    
    https://tronche.com/gui/x/icccm/sec-2.html#s-2.4 specifies:
> Once all the data in the selection has been retrieved,
> the requestor should delete the property in the SelectionNotify request
    
    Most Clipboard-Owners ignore whether or not the property is already set,
    so this is mostly a cosmetic change to keep the windows property list clean.
    
    However, at least synergy decides to wait for the requestor to delete
    the properties if they are already set by a previous paste (from synergy).
    
    Signed-off-by: Christoph Lohmann <20h_AT_r-36.net>

diff --git a/st.c b/st.c
index 27536d2..6736464 100644
--- a/st.c
+++ b/st.c
_AT_@ -1151,8 +1151,7 @@ selnotify(XEvent *e)
          * Deleting the property again tells the selection owner to send the
          * next data chunk in the property.
          */
- if (e->type == PropertyNotify)
- XDeleteProperty(xw.dpy, xw.win, (int)property);
+ XDeleteProperty(xw.dpy, xw.win, (int)property);
 }
 
 void
Received on Fri Jun 03 2016 - 15:03:51 CEST

This archive was generated by hypermail 2.3.0 : Fri Jun 03 2016 - 15:12:18 CEST