cgeo/c-geo-opensource — comment on commit 6bf07c2106c30503c011ae2c5cd4766906b8b092
File:  line 0
By rsudev:

I am not sure what your intentions have been with this change but I think it does not work as you intended.
For every map update still a new thread is created (a LoadThread) and (perhaps after the initial round) all the methods (DownloadThread.run(), DisplayThread.run()) are executed in this thread, so no thread re-using but getting back everything onto one thread (newly created per reqeuest) as it once has been sometime back in early 2011.
If this really was your intention, you should have moved the functionality from the other Thread classes into the LoadThread to make understandable whats going on.
I would ask you to revert this and discuss such fundamental changes in a separate issue/PR before putting them to master.