Skip to content
Snippets Groups Projects
Commit 0b1b1a6f authored by Rob Swindell's avatar Rob Swindell :speech_balloon:
Browse files

Fix issue with "Pause Logging" buttons not staying pressed in sbbsctrl-win32

Reported by Max (WESTLINE)...

Introduced in commit 63d0772d, since we are now calling the set_state()
callback periodically, and in sbbsctrl, it sets/resets the pause button
checked (depressed) state in response to this callback (should it?), this was
resetting the pause button checked/depressed state peridically (at the
configured sem file check interval, default 2 seconds).

The fix: if the new server state is the same as the previously set server
state, don't call the set_state() callback. I considered this as a potential
optimization when I first wrote that commit, but didn't see any obvious
downside to calling the set_state() callback redundantly. Now that I see a
downside, let's implement this saving/checking of the state to avoid
redundant "set" calls.
parent 5bea6c6b
No related branches found
No related tags found
No related merge requests found
Pipeline #5274 passed
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment