diff options
| author | dkf <donal.k.fellows@manchester.ac.uk> | 2010-01-05 09:44:53 (GMT) |
|---|---|---|
| committer | dkf <donal.k.fellows@manchester.ac.uk> | 2010-01-05 09:44:53 (GMT) |
| commit | dda6e2358ef98e30c75799b489a68a0805e391b2 (patch) | |
| tree | 6614253727e7042fe22d834df061463060456259 /generic | |
| parent | f1906c68efe43d621b82d612cb6126ff6718f9f2 (diff) | |
| download | tk-dda6e2358ef98e30c75799b489a68a0805e391b2.zip tk-dda6e2358ef98e30c75799b489a68a0805e391b2.tar.gz tk-dda6e2358ef98e30c75799b489a68a0805e391b2.tar.bz2 | |
[Bug 220950]: Don't delete the last menu entry when it is obvious that is not
what was intended.
Diffstat (limited to 'generic')
| -rw-r--r-- | generic/tkMenu.c | 15 |
1 files changed, 13 insertions, 2 deletions
diff --git a/generic/tkMenu.c b/generic/tkMenu.c index 98f3f60..7af304e 100644 --- a/generic/tkMenu.c +++ b/generic/tkMenu.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: tkMenu.c,v 1.41.2.3 2009/12/30 00:29:38 patthoyts Exp $ + * RCS: @(#) $Id: tkMenu.c,v 1.41.2.4 2010/01/05 09:44:54 dkf Exp $ */ /* @@ -777,7 +777,18 @@ MenuWidgetObjCmd( Tcl_WrongNumArgs(interp, 2, objv, "first ?last?"); goto error; } - if (TkGetMenuIndex(interp, menuPtr, objv[2], 0, &first) != TCL_OK) { + + /* + * If 'first' explicitly refers to past the end of the menu, we don't + * do anything. [Bug 220950] + */ + + if (isdigit(UCHAR(Tcl_GetString(objv[2])[0])) + && Tcl_GetIntFromObj(NULL, objv[2], &first) == TCL_OK) { + if (first >= menuPtr->numEntries) { + goto done; + } + } else if (TkGetMenuIndex(interp,menuPtr,objv[2],0,&first) != TCL_OK){ goto error; } if (objc == 3) { |