Improved queue

[ErikBjare]

Haven't done any testing to speak of, but I think this is an improvement.

Should improve reliability overall, thread should also cleanly exit on program close using atexit (which we should probably use in other places as well instead of handling signals).

[ErikBjare]

This has passed integration tests in ActivityWatch/activitywatch#82

[johan-bjareholt]

test/test_failqueue.py does not seem to work anymore?

[ErikBjare]

@johan-bjareholt It works for me now, please test it yourself.

[johan-bjareholt]

So we cannot queue insert_event anymore? That could potentially mess up aw-watcher-afk in set_state if aw-server is not accessible.

[ErikBjare]

@johan-bjareholt But wasn't aw-watcher-afk rewritten to use heartbeats? That should have removed the need for using insert_event, no?

Edit: I just checked and it seems to me that the use of insert_event there is no longer needed (provided we support receiving heartbeats with a duration, which we might not but probably should).

[johan-bjareholt]

Send event is still required! Heartbeats are only for updating the inserted events, and i use send_event every time we change state to be able to track the hibernation state (since you obviously cannot do heartbeats when the computer is hibernating).

[johan-bjareholt]

Pushed a fix in aw-watcher-afk straight to master so insert_event is no longer required.

ActivityWatch/aw-watcher-afk@ebaa72e

@ErikBjare Should we merge this now? Seems to work fine

[ErikBjare]

Great, merging.