search

andmcadams / iMessageForwarder

6 stars 1 forks source link

Infinite recursion #1

Closed andmcadams closed 4 years ago

andmcadams commented 4 years ago

Constantly resizing the window (dragging and never letting go) for a period of time will eventually cause the program to crash. This is because the resizing causes the functions _configure_canvas and _configure_scrollbars to get caught in an infinite loop. Removing the call to _configure_scrollbars in _configure_canvas seems to stop this issue, but it needs further testing to make sure it doesn't adversely impact other features.

andmcadams commented 4 years ago

Removing the call causes other issues. A fix looks like it would involve rewriting the handlers for canvas resizing and scrollbar handling, which should happen anyway.

andmcadams commented 4 years ago

Appears to have been solved through refactoring.