[Marble-devel] Review Request: Threadsafe HttpDownloadManager

Henry de Valence hdevalence at gmail.com
Fri Aug 7 02:01:07 CEST 2009


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/1235/
-----------------------------------------------------------

Review request for marble.


Summary
-------

Adds threadsafety to the HttpDownloadManager using a QMutex.


Diffs
-----

  trunk/KDE/kdeedu/marble/src/lib/HttpDownloadManager.h 1008141 
  trunk/KDE/kdeedu/marble/src/lib/HttpDownloadManager.cpp 1008141 

Diff: http://reviewboard.kde.org/r/1235/diff


Testing
-------

Tested using the downloader to pull OSM tiles but didn't test from another thread, because I'm not sure how to go about doing that yet. I have some changes to the MarbleRunners that will require it, but they're not done yet.


Thanks,

Henry



More information about the Marble-devel mailing list