In d2common/d2datautils we should have unit tests for all methods of StreamReader and StreamWriter.

Type of Issues (Enhancement, Error, Bug, Question)

Bug:

1. Initialize Window
2. Show it (window.Read())
3. Run window.Disable()
4. The whole desktop environment freezes, nothing is clickable anymore
5. Change to tty terminal and kill application
6. Things are normal again

Operating System

Ubuntu 16.04 LTS 64 bit

Python version

Python 3.5.2

PySi

Describe the bug
An Exception thrown in FinishLoadScene will cause loadingSceneAsync to not be cleared. This means FinishLoadScene will be called every frame assuming the error keeps being throw.

To Reproduce

1. Throw inside one of the FinishLoadScene override methods
2. Load a Scene on the server using ServerChangeScene
3. FinishLoadScene methods be called every frame

Describe the feature you'd like
At the bottom of loadResearch, after research.json and pre-reqs are loaded + processed, it would be nice to have a pass that detects cyclic research dependencies (i.e. in RESEARCH.pPRList).

Describe why do you think it is needed
It's currently possible to accidentally create research dependency cycles. This is obviously no good.

Detecting this,

Add a filter to select a subset of threads.
Use SearchBox control with DelayedTextChanged event.
Hide all of the threads on the Timeline not containing specified string (should be case-insensitive).
Use space next to the Header row on the Timelne for the search box.

Code in: ThreadView.xaml.cs

https://codepo8.github.io/css-fork-on-github-ribbon/

Make it more obvious that people can contribute to this list via github. I think the current text-only link in the description is easy to skip over and miss.

(As this is an enhancement not a bug, I don't think i need to provide too much information.)
Although enemies in Super Mario don't have such animation either, as a more elegant platform game, it is better to add animations like kicking Tux when Tux collides them, because it can make the enemies more vivid.

From the comment in SDL_filesystem.h:
Please call SDL_free() on the pointer when you are done with it

When building the project with MSVC, I ran into some debug heap asserts early into the program startup.
The culprit appears to be the code in m_config.c that does free(prefdir) as changing these to SDL_free(prefdir) resolved the issue.

I'm not sure what to do about `GetDefaultConfi

Context

There is a use case for creating a timer that has a variable duration. For example, if you wanted to randomly space events during a game.

Proposal

Add the ability to specify a range (min and max) of time and allow the timer to randomly choose a duration within that range.