summaryrefslogtreecommitdiffstats
path: root/win/tkWinWm.c
diff options
context:
space:
mode:
authormdejong <mdejong>2004-10-29 22:28:30 (GMT)
committermdejong <mdejong>2004-10-29 22:28:30 (GMT)
commitd71f8610d6bfae2495ad9fdcdb996139b53771f5 (patch)
treec52ced77c155992f8ae1b74ac0502013b47a3acc /win/tkWinWm.c
parent79de997b6c48f81b2ac3b1094b97bf677251115d (diff)
downloadtk-d71f8610d6bfae2495ad9fdcdb996139b53771f5.zip
tk-d71f8610d6bfae2495ad9fdcdb996139b53771f5.tar.gz
tk-d71f8610d6bfae2495ad9fdcdb996139b53771f5.tar.bz2
* tests/wm.test: Add Win32 test cases for attributes
subcommand. * win/tkWinWm.c (WmAttributesCmd): Fixup broken option processing logic for attributes subcommand.
Diffstat (limited to 'win/tkWinWm.c')
-rw-r--r--win/tkWinWm.c20
1 files changed, 11 insertions, 9 deletions
diff --git a/win/tkWinWm.c b/win/tkWinWm.c
index 4a57d6a..a00d0ad 100644
--- a/win/tkWinWm.c
+++ b/win/tkWinWm.c
@@ -12,7 +12,7 @@
* See the file "license.terms" for information on usage and redistribution
* of this file, and for a DISCLAIMER OF ALL WARRANTIES.
*
- * RCS: @(#) $Id: tkWinWm.c,v 1.78 2004/10/21 01:13:06 hobbs Exp $
+ * RCS: @(#) $Id: tkWinWm.c,v 1.79 2004/10/29 22:28:31 mdejong Exp $
*/
#include "tkWinInt.h"
@@ -2859,7 +2859,7 @@ WmAttributesCmd(tkwin, winPtr, interp, objc, objv)
char *string;
int i, boolean, length;
- if (objc < 3) {
+ if ((objc < 3) || ((objc > 5) && ((objc%2) == 0))) {
configArgs:
Tcl_WrongNumArgs(interp, 2, objv,
"window"
@@ -2902,12 +2902,12 @@ WmAttributesCmd(tkwin, winPtr, interp, objc, objv)
} else if (strncmp(string, "-alpha", length) == 0) {
stylePtr = &exStyle;
styleBit = WS_EX_LAYERED;
- } else if ((strncmp(string, "-toolwindow", length) == 0)
- && (length >= 3)) {
+ } else if ((length > 3)
+ && (strncmp(string, "-toolwindow", length) == 0)) {
stylePtr = &exStyle;
styleBit = WS_EX_TOOLWINDOW;
- } else if ((strncmp(string, "-topmost", length) == 0)
- && (length >= 3)) {
+ } else if ((length > 3)
+ && (strncmp(string, "-topmost", length) == 0)) {
stylePtr = &exStyle;
styleBit = WS_EX_TOPMOST;
if ((i < objc-1) && (winPtr->flags & TK_EMBEDDED)) {
@@ -2922,7 +2922,7 @@ WmAttributesCmd(tkwin, winPtr, interp, objc, objv)
if (styleBit == WS_EX_LAYERED) {
double dval;
- if (i == objc-1) {
+ if (objc == 4) {
Tcl_SetDoubleObj(Tcl_GetObjResult(interp), wmPtr->alpha);
} else {
if ((i < objc-1) &&
@@ -2964,7 +2964,7 @@ WmAttributesCmd(tkwin, winPtr, interp, objc, objv)
!= TCL_OK)) {
return TCL_ERROR;
}
- if (i == objc-1) {
+ if (objc == 4) {
Tcl_SetIntObj(Tcl_GetObjResult(interp),
((*stylePtr & styleBit) != 0));
} else if (boolean) {
@@ -4826,8 +4826,10 @@ WmStateCmd(tkwin, winPtr, interp, objc, objv)
} else if (index == OPT_WITHDRAWN) {
wmPtr->flags |= WM_WITHDRAWN;
TkpWmSetState(winPtr, WithdrawnState);
- } else { /* OPT_ZOOMED */
+ } else if (index == OPT_ZOOMED) {
TkpWmSetState(winPtr, ZoomState);
+ } else {
+ Tcl_Panic("wm state not matched");
}
} else {
if (wmPtr->iconFor != NULL) {