[Marble-devel] Review Request: Purge Download Queues when switching to offline

Bernhard Beschow bbeschow at cs.tu-berlin.de
Thu Feb 2 23:32:50 UTC 2012


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


What happens if downloading is (re-)enabled? IMO it should try to update blurry tiles by triggering TextureLayer::reset().

- Bernhard Beschow


On Feb. 2, 2012, 8:48 p.m., Thibaut Gridel wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/103853/
> -----------------------------------------------------------
> 
> (Updated Feb. 2, 2012, 8:48 p.m.)
> 
> 
> Review request for Marble and Bernhard Beschow.
> 
> 
> Description
> -------
> 
> Until now, switching to offline mode kept the download queues filled, meaning that really going offline would make 3 retries of all those download requests.
> This patch removes this, and cleans redundant list of current downloads from TileManager, as the DownloadManager simply discards tasks when offline anyway.
> 
> 
> Diffs
> -----
> 
>   src/lib/DownloadQueueSet.h 9d024f9 
>   src/lib/DownloadQueueSet.cpp ad4a777 
>   src/lib/HttpDownloadManager.cpp 262971f 
>   src/lib/TileLoader.h aef7e84 
>   src/lib/TileLoader.cpp e938bea 
> 
> Diff: http://git.reviewboard.kde.org/r/103853/diff/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Thibaut Gridel
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/marble-devel/attachments/20120202/d0bb804f/attachment.html>


More information about the Marble-devel mailing list