Allow re-downloading of blank/corrupt/old tiles. It downloads and saves them, but then you have multiple copies of the same tile and it just uses the first one, so database grows, but effect is that tile was never downloaded. Fix (already coded and tested) is to simply delete all old matching tiles as you add the updated tile. A minor change is to make ServerOnly save downloaded tiles to the database. So from this change, all tiles downloaded with any option are in the database. This is now another way (besides rip map) to refresh any blank/corrupt/old areas of the map. Also fixed several occurrences of “QMutex: destroying locked mutex“ error message where a lock was never unlocked. Also fixed one case where lockforread was used when lockforwrite should be. Also fixed apparent misuse of locks. If they are not a member of a long lifetime object, they should be static or they have zero effect.
Allow re-downloading of blank/corrupt/old tiles. It downloads and saves them, but then you have multiple copies of the same tile and it just uses the first one, so database grows, but effect is that tile was never downloaded. Fix (already coded and tested) is to simply delete all old matching tiles as you add the updated tile.
A minor change is to make ServerOnly save downloaded tiles to the database. So from this change, all tiles downloaded with any option are in the database. This is now another way (besides rip map) to refresh any blank/corrupt/old areas of the map.
Also fixed several occurrences of “QMutex: destroying locked mutex“ error message where a lock was never unlocked.
Also fixed one case where lockforread was used when lockforwrite should be.
Also fixed apparent misuse of locks. If they are not a member of a long lifetime object, they should be static or they have zero effect.