Skip to content

Commit

Permalink
gui: Remove obsolete check_for_multiple_listbox_events_bug
Browse files Browse the repository at this point in the history
Formerly in vars.tcl, call and message previously commented out.

As per John, "looks like I fixed that bug"

Signed-Off-By: Jeff Kletsky <[email protected]>
  • Loading branch information
jeffsf authored and Mimoja committed Apr 15, 2021
1 parent 8c62605 commit 21d373e
Showing 1 changed file with 0 additions and 22 deletions.
22 changes: 0 additions & 22 deletions de1plus/vars.tcl
Original file line number Diff line number Diff line change
Expand Up @@ -2146,24 +2146,6 @@ proc fill_advanced_profile_steps_listbox {} {
update idletasks
}

# on androwish some listbox selctions are causing multiple cascading events, and we don't know why
# this is a work around that assumes that each cascading event will happen within 100ms of each others
set time_of_last_listbox_event [clock milliseconds]
proc check_for_multiple_listbox_events_bug {} {
msg -DEBUG "::de1(current_context) $::de1(current_context)"
return 0

set now [clock milliseconds]
set diff [expr {$now - $::time_of_last_listbox_event}]
set ::time_of_last_listbox_event $now

if {$diff < 100} {
return 1
}

return 0
}

proc load_language {} {
set stepnum [$::languages_widget curselection]
if {$stepnum == ""} {
Expand All @@ -2187,10 +2169,6 @@ proc load_advanced_profile_step {{force 0}} {
return
}

#if {[check_for_multiple_listbox_events_bug] == 1} {
# return
#}

set stepnum [$::advanced_shot_steps_widget curselection]
if {$stepnum == ""} {
#set stepnum
Expand Down

0 comments on commit 21d373e

Please sign in to comment.