revert -r1.45 of group.c (log was: fix window name and class to match

cwmrc(5)).  instead, fix cwmrc(5) to match the old behavior which also
happens to match the example config, of which many have based their
configs; this also nicely matches the output of xprop(1).

clean-up of variable names as a separate commit.

suggested by sthen (and something we should have done initially).

discussed with and ok oga@
This commit is contained in:
okan 2010-07-30 12:28:19 +00:00
parent 184d7c3b41
commit bd36084da7
2 changed files with 10 additions and 9 deletions

View File

@ -1,4 +1,4 @@
.\" $OpenBSD: cwmrc.5,v 1.30 2010/07/23 15:13:00 okan Exp $
.\" $OpenBSD: cwmrc.5,v 1.31 2010/07/30 12:28:19 okan Exp $
.\"
.\" Copyright (c) 2004,2005 Marius Aamodt Eriksen <marius@monkey.org>
.\"
@ -14,7 +14,7 @@
.\" ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF
.\" OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
.\"
.Dd $Mdocdate: July 23 2010 $
.Dd $Mdocdate: July 30 2010 $
.Dt CWMRC 5
.Os
.Sh NAME
@ -27,8 +27,8 @@ configuration file.
The following options are accepted in the configuration file:
.Pp
.Bl -tag -width Ds -compact
.It Ic autogroup Ar group windowclass
.It Ic autogroup Ar group windowclass,windowname
.It Ic autogroup Ar group windowname
.It Ic autogroup Ar group windowname,windowclass
Control automatic window grouping, based on the class and/or name
properties, where
.Ar group
@ -206,8 +206,8 @@ command xmms xmms
command top "xterm -e top"
# Autogroup definitions
autogroup 3 "XTerm,aterm"
autogroup 3 "XTerm,xterm"
autogroup 3 "aterm,XTerm"
autogroup 3 "xterm,XTerm"
# Ignore programs by that name by not drawing borders around them.
ignore XMMS

View File

@ -16,7 +16,7 @@
* ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF
* OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
*
* $Id: group.c,v 1.45 2010/06/28 12:29:04 okan Exp $
* $Id: group.c,v 1.46 2010/07/30 12:28:19 okan Exp $
*/
#include <sys/param.h>
@ -188,11 +188,12 @@ group_make_autogroup(struct conf *conf, char *class, int no)
if ((p = strchr(class, ',')) == NULL) {
aw->name = NULL;
aw->class = xstrdup(class);
} else {
*(p++) = '\0';
aw->name = xstrdup(p);
aw->name = xstrdup(class);
aw->class = xstrdup(p);
}
aw->class = xstrdup(class);
aw->num = no;
TAILQ_INSERT_TAIL(&conf->autogroupq, aw, entry);