search

Charcoal-SE / PulseMonitor

PulseMonitor is the chat interface to Halflife.
MIT License
4 stars 3 forks source link

unnotify sometimes doesn't work: list.remove(x): x not in list #28

Closed tripleee closed 4 years ago

tripleee commented 4 years ago

This is an old bug actually, I tried to troubleshoot it several months ago and deployed some diagnostics to find the root cause ... which then caused the problem to disappear.

Now I rolled back those changes a few weeks ago, and sure enough, the problem reappeared.

https://chat.stackexchange.com/transcript/message/53704619#53704619

tripleee commented 4 years ago

Martijn was graceful enough to have a look at the code, and found the problem pretty quickly:

https://chat.stackexchange.com/transcript/message/53704759#53704759

Also, some Pythonicity advice.