Re: 1.4.7-beta: new features

The diff between Axel's sloppy and mainstream sloppy:

--- sloppy.c.old        2013-06-05 11:36:50.142790834 -0300
+++ sloppy.c    2013-06-05 11:39:04.073786877 -0300
@@ -103,8 +103,8 @@ main (void)
         char shell[256];

         snprintf (shell, sizeof(shell),
-                  "$RATPOISON -c \"select `$RATPOISON -c 'windows %%i %%n'"
-                  "| awk '$1 == %ld { print $2; }'`\"",
+                  "$RATPOISON -c \"select $($RATPOISON -c 'windows %%i %%n
%%f'"
+                  "| awk '$1 == %ld && $3 != '$($RATPOISON -c curframe)'
&& $3 != \"\" {print $2}')\" 2>/dev/null",
                   event.xcrossing.window);
         spawn (shell);
         wait (NULL);




On Wed, Jun 5, 2013 at 10:46 AM, Ellington Santos <[email protected]>wrote:

> Axel,
>
> sloppy works fine with two monitors with xrandr:
>
> $ sloppy
> $ xrandr --output LVDS1 --auto --primary --output VGA1 --auto --right-of
> LVDS1
>
> Thanks.
>
> Ellington
>
>
>
>
> On Wed, Jun 5, 2013 at 9:58 AM, Axel Beckert <[email protected]> wrote:
>
>> Hi,
>>
>> On Wed, Jun 05, 2013 at 09:47:49AM -0300, Ellington Santos wrote:
>> > On Wed, Jun 5, 2013 at 4:20 AM, Axel Beckert <[email protected]>
>> wrote:
>> > > On Wed, Jun 05, 2013 at 08:15:11AM +0200, Jérémie Courrèges-Anglas
>> wrote:
>> > > > I cleaned up unrat.c and sloppy.c a bit yesterday evening.  Which
>> lead
>> > > > me to test them.  Is there anyone here that *actually* uses them? :)
>> > >
>> > > I do, in a slightly modified version from two years ago:
>> > > https://github.com/xtaran/ratpoison-desktop/tree/master/sloppy
>> >
>> > Axel's version corrects a bug when changing a one-frame window to
>> another
>> > with mouse pointer inside frame.
>>
>> I did these fixes quite some time ago so that I don't remember much
>> details anymore, so all explanations I can offer are my git commit
>> messages and the code itself. :-)
>>
>> Nevertheless I can rebase my changes onto ratpoison's git repo and
>> offer that for merging or cherry-picking if wanted.
>>
>> > I will test sloppy with two monitors today.
>>
>> Haven't done that for quite a while. IIRC ratpoison didn't work
>> properly if the second screen was added via xrandr while it IIRC
>> worked fine if the two screens were separate displays.
>>
>>                 Kind regards, Axel
>> --
>> /~\  Plain Text Ribbon Campaign                   | Axel Beckert
>> \ /  Say No to HTML in E-Mail and News            | [email protected] (Mail)
>>  X   See http://www.asciiribbon.org/              | [email protected](Mail+Jabber)
>> / \  I love long mails: http://email.is-not-s.ms/ | http://noone.org/abe/(Web)
>>
>> _______________________________________________
>> Ratpoison-devel mailing list
>> [email protected]
>> https://lists.nongnu.org/mailman/listinfo/ratpoison-devel
>>
>
>

_______________________________________________
Ratpoison-devel mailing list
[email protected]
https://lists.nongnu.org/mailman/listinfo/ratpoison-devel

--- sloppy.c.old	2013-06-05 11:36:50.142790834 -0300
+++ sloppy.c	2013-06-05 11:39:04.073786877 -0300
@@ -103,8 +103,8 @@ main (void)
         char shell[256];
 
         snprintf (shell, sizeof(shell),
-                  "$RATPOISON -c \"select `$RATPOISON -c 'windows %%i %%n'"
-                  "| awk '$1 == %ld { print $2; }'`\"",
+                  "$RATPOISON -c \"select $($RATPOISON -c 'windows %%i %%n %%f'"
+                  "| awk '$1 == %ld && $3 != '$($RATPOISON -c curframe)' && $3 != \"\" {print $2}')\" 2>/dev/null",
                   event.xcrossing.window);
         spawn (shell);
         wait (NULL);