diff options
author | David Kalnischkies <david@kalnischkies.de> | 2017-12-13 21:39:16 +0100 |
---|---|---|
committer | David Kalnischkies <david@kalnischkies.de> | 2017-12-13 23:53:41 +0100 |
commit | 1adcf56bec7d2127d83aa423916639740fe8e586 (patch) | |
tree | 39802989990796e903d8e670b07408380b117c21 /apt-private | |
parent | 957381a0d26ec11a172ebfc64f892d1b31f0c193 (diff) |
avoid some useless casts reported by -Wuseless-cast
The casts are useless, but the reports show some where we can actually
improve the code by replacing them with better alternatives like
converting whatever int type into a string instead of casting to a
specific one which might in the future be too small.
Reported-By: gcc -Wuseless-cast
Diffstat (limited to 'apt-private')
-rw-r--r-- | apt-private/private-source.cc | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/apt-private/private-source.cc b/apt-private/private-source.cc index 47610cd80..32651cfdb 100644 --- a/apt-private/private-source.cc +++ b/apt-private/private-source.cc @@ -208,12 +208,7 @@ static pkgSrcRecords::Parser *FindSrc(const char *Name, // or RelTag if (Cache.BuildPolicy() == false) return nullptr; - pkgPolicy * Policy = dynamic_cast<pkgPolicy*>(Cache.GetPolicy()); - if (Policy == nullptr) - { - _error->Fatal("Implementation error: dynamic up-casting policy engine failed in FindSrc!"); - return nullptr; - } + pkgPolicy * const Policy = Cache.GetPolicy(); pkgCache::VerIterator const Ver = Policy->GetCandidateVer(Pkg); if (Ver.end() == false) { |