From: SourceForge.net <no...@so...> - 2006-05-25 23:53:01
|
Bugs item #1224330, was opened at 2005-06-20 11:35 Message generated for change (Comment added) made by hobbs You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=112997&aid=1224330&group_id=12997 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: 13. Win Menus Group: current: 8.4.13 >Status: Closed >Resolution: Fixed Priority: 8 Submitted By: fvogelnew1 (fvogelnew1) Assigned to: Jeffrey Hobbs (hobbs) Summary: Bug in menu delete Initial Comment: The following code shows a bug in the windows menu implementation: menu .menu for {set i 0} {$i<20} {incr i} { .menu add command -label "Entry $i" -command \ "tk_messageBox -message \"Entry $i!\"" } .menu delete 10 pack [menubutton .m -menu .menu -text Menu] Menu entries below the deleted item are not selectable. If clicked, the last entry before the deleted one is executed. I submit this bug as a new one because I don't know how to reopen bug 525111. This happens on Win XP SP2 with the ActiveState binary distribution of Tcl/Tk 8.4.10. ---------------------------------------------------------------------- >Comment By: Jeffrey Hobbs (hobbs) Date: 2006-05-25 16:52 Message: Logged In: YES user_id=72656 This turned out to be an optimizer bug in VC6. I have adjusted the code to correct this. It was actually in generic Tcl code, but only the Windows compiler was affected. Fixed in 8.4.14 and 8.5a4. ---------------------------------------------------------------------- Comment By: Jeffrey Hobbs (hobbs) Date: 2006-05-25 11:52 Message: Logged In: YES user_id=72656 I can hit the problem in 8.4 and 8.5, but only on optimized builds. Debug builds don't show the problem ... state init problem? Race condition? ---------------------------------------------------------------------- Comment By: fvogelnew1 (fvogelnew1) Date: 2006-04-27 14:11 Message: Logged In: YES user_id=1245417 Hmmm, something must have gone wrong. I remember having checked on the source code that this was fixed (see below), but now using the precompiled activestate binaries the same problem shows up again. Might have been broken again between our checks and the final 8.4.13 release. Reopening this bug. Francois ---------------------------------------------------------------------- Comment By: fvogelnew1 (fvogelnew1) Date: 2006-03-30 22:19 Message: Logged In: YES user_id=1245417 Yes, you're right Jeff. It's broken in 8.4.12 but works with the core-8.4.13 sources. I close this, thanks for having checked. Cheers, Francois ---------------------------------------------------------------------- Comment By: Jeffrey Hobbs (hobbs) Date: 2006-03-30 17:02 Message: Logged In: YES user_id=72656 This works in my sandbox for 8.4.13. ---------------------------------------------------------------------- Comment By: fvogelnew1 (fvogelnew1) Date: 2006-03-18 05:06 Message: Logged In: YES user_id=1245417 Still broken with 8.4.12. Works however with cvs HEAD from today. Could we backport the fix to the 8.4 branch? Francois ---------------------------------------------------------------------- Comment By: fvogelnew1 (fvogelnew1) Date: 2005-06-20 13:01 Message: Logged In: YES user_id=1245417 Note however that if the menu is torn off, behavior is apparently OK. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=112997&aid=1224330&group_id=12997 |