On Tue, 10 Aug 2010 08:20:34 +0200, Michael Jennings <m...@kainx.org> wrote:
> On Monday, 09 August 2010, at 22:27:00 (+0200), > Kim Woelders wrote: > >> I was thinking something like the attached patch. > > Looks good to me. Would you mind committing? > >> Then I found XA_CLIPBOARD(Xdisplay). Assuming we have X11/Xmu/Atoms.h, >> which I think practically always is true, XA_CLIPBOARD(Xdisplay) will >> refer to the "CLIPBOARD" atom in screen.c (used in IS_SELECTION()) where >> X11/Xmu/Atoms.h is included before screen.h, whereas >> XA_CLIPBOARD(Xdisplay) will refer to the "PRIMARY" atom in script.c and >> term.c due to definition in screen.h. >> It is of course simple to add props[PROP_CLIPBOARD] too, but I'm not >> sure >> what the intended behavior is. > > The automated behavior should always interact with PRIMARY. The only > way Eterm should ever touch CLIPBOARD is if a script directly > references it. This is due to the fact that Eterm's copy is always > implicit (again, outside of a script). > > If you have time, feel free to add the clipboard props as well. :) > Ok, done :) As per commit message this causes a few changes. I guess the change is good in script.c, not sure about term.c. /Kim ------------------------------------------------------------------------------ This SF.net email is sponsored by Make an app they can't live without Enter the BlackBerry Developer Challenge http://p.sf.net/sfu/RIM-dev2dev _______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
