[Fvwm-crystal-commits] SF.net SVN: fvwm-crystal:[1046] fvwm/components/functions

[<dom...@us...>]

Revision: 1046
          http://sourceforge.net/p/fvwm-crystal/code/1046
Author:   dominique_libre
Date:     2020-10-06 23:33:31 +0000 (Tue, 06 Oct 2020)
Log Message:
-----------
Exec-Accelerator: fix Restart by removing superflous quoting

Modified Paths:
--------------
    ChangeLog
    fvwm/components/functions/Exec-Accelerator

Modified: ChangeLog
===================================================================
--- ChangeLog	2020-10-06 23:18:49 UTC (rev 1045)
+++ ChangeLog	2020-10-06 23:33:31 UTC (rev 1046)
@@ -6,6 +6,8 @@
   fvwm/components/styles/FVWM: revert trayer style in revision 898
   due to trayer moving with the "arrange all windows" button;
   thanks to Gianni Ceccarelli for reporting this.
+  fvwm/components/functions/Exec-Accelerator: fix Restart by removing
+  superflous quoting.
 
 jeudi 17 septembre 2020
 + fvwm/locale/zh_CN*: Add chinese translation. Thanks to 玉堂白鹤

Modified: fvwm/components/functions/Exec-Accelerator
===================================================================
--- fvwm/components/functions/Exec-Accelerator	2020-10-06 23:18:49 UTC (rev 1045)
+++ fvwm/components/functions/Exec-Accelerator	2020-10-06 23:33:31 UTC (rev 1046)
@@ -49,11 +49,12 @@
 
 # If urxvtc is killed, urxvtd is killed too, so start urxvtd if needed.
 # StartUrxvtc need an argument or urxvtc will fail; see the A, AA and AT functions.
+# With superflous argument quoting, fvwm work fine after startup, but not sometime after Restart.
 DestroyFunc StartUrxvtc
 AddToFunc StartUrxvtc
-+ I Piperead 'urxvtc "$*" $[infostore.SILENT]; if [ $? = 2 ]; then \
++ I Piperead 'urxvtc $* $[infostore.SILENT]; if [ $? = 2 ]; then \
 				urxvtd -f -o $[infostore.SILENT]; \
-				urxvtc "$*"; \
+				urxvtc $*; \
 			fi'
 
 DestroyFunc StartTerminal
@@ -70,22 +71,22 @@
 DestroyFunc A
 AddToFunc A
 + I Window-SetPlacementPolicy
-+ I Test (EnvMatch infostore.DefaultTerminal urxvtc) StartUrxvtc "-e $*"
++ I Test (EnvMatch infostore.DefaultTerminal urxvtc) StartUrxvtc -e $*
 + I TestRc (NoMatch) StartTerminal "$*"
 
 DestroyFunc AA
 AddToFunc AA
 + I Window-SetPlacementPolicy
-+ I Test (EnvMatch infostore.DefaultTerminal urxvtc) StartUrxvtc "-name urxvt"
++ I Test (EnvMatch infostore.DefaultTerminal urxvtc) StartUrxvtc -name urxvt
 + I TestRc (NoMatch) Exec exec $[infostore.DefaultTerminal] $[infostore.SILENT]
 
 # Usage: AT [title] [command] (Use quotes if you want command options or title with spaces)
 # !: doesn't work anymore with gnome-terminal due to a bug in gnome-terminal (doesn't honor anymore it's own --title option (and --name and --class too)).
-# As the preceding similar bug tooks years to be fixed and that one was introduced only a few months later, just choose another terminal
+# The preceding similar bug tooks years to be fixed, and a new one was introduced only a few months later, better just choose another terminal
 DestroyFunc AT
 AddToFunc AT
 + I Window-SetPlacementPolicy
-+ I Test (EnvMatch infostore.DefaultTerminal urxvtc) StartUrxvtc "--title "$0" -name "$0" -e $[1-]"
++ I Test (EnvMatch infostore.DefaultTerminal urxvtc) StartUrxvtc --title "$0" -name "$0" -e $[1-]
 + I TestRc (NoMatch) StartTitledTerminal "$0" $[1-]
 
 # Usage: C [command] [command options]

This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.