Fixes

[JuiP]

• Fix Warning: Source ID 108 was not found when attempting to remove it
• Port from GObject to GLib

@quozl @chimosky @Saumya-Mishra9129 Kindly test, review and merge :)

[Saumya-Mishra9129]

Fix Warning: Source ID 108 was not found when attempting to remove it

I wonder why I am not getting any such warning. 😕

[JuiP]

Way to reproduce it -
Start the activity, type something in the text box, now click on new game button; wait till new set of images are displayed. I found the warning in the logs after this.

[quozl]

• it is more correct to clear the timeout id after source_remove in _all_clear,
• i'm not sure if all the rules of dealing with source ids are being followed; see for example _dance_step which creates a new source id without checking if the previous one is removed; almost as if a different variable name should be used for the separate timer.

[JuiP]

@quozl Thanks! I hope c9e6923 fixes it :)

[quozl]

• _autonext is called by both the GLib event-loop and by other code; an overload of semantics ... when it is called by autoplay are you sure it should clear the _timeout_id?
• _dance_step is also called by both the GLib event-loop and by other code; an overload of semantics ... it does not clear the _timeout_id when the counter is not less than 10, yet by returning False (the default) the timeout will have been removed automatically,
• it is not necessary to set _timeout_id to None immediately before a call to GLib.timeout_add that assigns the source ID ot the _timeout_id variable.

I remain concerned about the use of the same variable name for multiple timeouts.