Replace wxTimer in SCH_EDITOR_CONTROL by timestamp

Bug #1830202 reported by Simon Richter
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
KiCad
Fix Released
Unknown

Bug Description

In SCH_EDITOR_CONTROL, there is a timer that is started after a search, and if another search is started while the timer is still running, the search may wrap to the start.

No action is taken or needed when the timer elapses, so this could be done a bit more lightweight by saving the event timestamp from the last search and comparing the timestamp from the current search.

In addition, it would be nice if that mechanism wouldn't use a global object.

Offending line:

eeschema/tools/sch_editor_control.cpp:213: static wxTimer wrapAroundTimer;

tags: added: starter
removed: newbie-friendly
Seth Hillbrand (sethh)
Changed in kicad:
status: New → Triaged
importance: Undecided → Wishlist
Revision history for this message
KiCad Janitor (kicad-janitor) wrote :

KiCad bug tracker has moved to Gitlab. This report is now available here: https://gitlab.com/kicad/code/kicad/-/issues/2430

Changed in kicad:
status: Triaged → Expired
Changed in kicad:
importance: Wishlist → Unknown
status: Expired → New
Changed in kicad:
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.