RedAuburn/organicmaps
1
Fork 0
forked from organicmaps/organicmaps
Code Pull requests Activity

Review fixes.

Browse source
This commit is contained in:
Yuri Gorshenin 2015-11-02 13:16:24 +03:00
parent 2dd63cac9b
commit 735cfe3c8c
2 changed files with 16 additions and 16 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

28
storage/storage.cpp
View file

@ -719,41 +719,41 @@ TStatus Storage::CountryStatusFull(TIndex const & index, TStatus const status) c
return TStatus::EOnDisk;
}
MapOptions Storage::NormalizeDownloadFileSet(TIndex const & index, MapOptions opt) const
MapOptions Storage::NormalizeDownloadFileSet(TIndex const & index, MapOptions options) const
{
auto const & country = GetCountryFile(index);
// Car routing files are useless without map files.
if (HasOptions(opt, MapOptions::CarRouting))
opt = SetOptions(opt, MapOptions::Map);
if (HasOptions(options, MapOptions::CarRouting))
options = SetOptions(options, MapOptions::Map);
TLocalFilePtr localCountryFile = GetLatestLocalFile(index);
for (MapOptions file : {MapOptions::Map, MapOptions::CarRouting})
for (MapOptions option : {MapOptions::Map, MapOptions::CarRouting})
{
// Check whether requested files are on disk and up-to-date.
if (HasOptions(opt, file) && localCountryFile && localCountryFile->OnDisk(file) &&
if (HasOptions(options, option) && localCountryFile && localCountryFile->OnDisk(option) &&
localCountryFile->GetVersion() == GetCurrentDataVersion())
{
opt = UnsetOptions(opt, file);
options = UnsetOptions(options, option);
}
// Check whether requested file is not empty.
if (GetRemoteSize(country, file) == 0)
if (GetRemoteSize(country, option) == 0)
{
ASSERT_NOT_EQUAL(MapOptions::Map, file, ("Map can't be empty."));
opt = UnsetOptions(opt, file);
ASSERT_NOT_EQUAL(MapOptions::Map, option, ("Map can't be empty."));
options = UnsetOptions(options, option);
}
}
return opt;
return options;
}
MapOptions Storage::NormalizeDeleteFileSet(MapOptions opt) const
MapOptions Storage::NormalizeDeleteFileSet(MapOptions options) const
{
// Car routing files are useless without map files.
if (HasOptions(opt, MapOptions::Map))
opt = SetOptions(opt, MapOptions::CarRouting);
return opt;
if (HasOptions(options, MapOptions::Map))
options = SetOptions(options, MapOptions::CarRouting);
return options;
}
QueuedCountry * Storage::FindCountryInQueue(TIndex const & index)

4
storage/storage.hpp
View file

@ -190,11 +190,11 @@ private:
// Modifies file set of requested files - always adds a map file
// when routing file is requested for downloading, but drops all
// already downloaded and up-to-date files.
MapOptions NormalizeDownloadFileSet(TIndex const & index, MapOptions opt) const;
MapOptions NormalizeDownloadFileSet(TIndex const & index, MapOptions options) const;
// Modifies file set of file to deletion - always adds (marks for
// removal) a routing file when map file is marked for deletion.
MapOptions NormalizeDeleteFileSet(MapOptions opt) const;
MapOptions NormalizeDeleteFileSet(MapOptions options) const;
// Returns a pointer to a country in the downloader's queue.
QueuedCountry * FindCountryInQueue(TIndex const & index);