commit 60aeb37edb8c5280d31b6b3c801d09c7a5fdca76
Author:     v4hn <m...@v4hn.de>
AuthorDate: Thu May 19 12:16:57 2016 +0200
Commit:     Christoph Lohmann <2...@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 <2...@r-36.net>

diff --git a/st.c b/st.c
index 27536d2..6736464 100644
--- a/st.c
+++ b/st.c
@@ -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

Reply via email to