diff options
| author | Julian Andres Klode <jak@debian.org> | 2016-09-28 00:49:45 +0200 |
|---|---|---|
| committer | Julian Andres Klode <jak@debian.org> | 2016-11-22 22:58:33 +0100 |
| commit | 49521f87851089bdb4097b715d09a8bd348aa60a (patch) | |
| tree | 7cedcd05be67505f4949eb18dbe155178b76e4bb /apt-pkg/edsp/edsplistparser.h | |
| parent | c0723bf1a60daf45096998d4ae9feee3c44343f8 (diff) | |
Do not use MD5SumValue for Description_md5()
Our profile says we spend about 5% of the time transforming the
hex digits into the binary format used by HashsumValue, all for
comparing them against the other strings. That makes no sense
at all.
According to callgrind, this reduces the overall instruction
count from 5,3 billion to 5 billion in my example, which
roughly matches the 5%.
Diffstat (limited to 'apt-pkg/edsp/edsplistparser.h')
| -rw-r--r-- | apt-pkg/edsp/edsplistparser.h | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/apt-pkg/edsp/edsplistparser.h b/apt-pkg/edsp/edsplistparser.h index 84138d6a8..4904b6567 100644 --- a/apt-pkg/edsp/edsplistparser.h +++ b/apt-pkg/edsp/edsplistparser.h @@ -24,12 +24,15 @@ #include <apt-pkg/tagfile.h> #endif +namespace APT { + class StringView; +} class APT_HIDDEN edspLikeListParser : public debListParser { public: virtual bool NewVersion(pkgCache::VerIterator &Ver) APT_OVERRIDE; virtual std::vector<std::string> AvailableDescriptionLanguages() APT_OVERRIDE; - virtual MD5SumValue Description_md5() APT_OVERRIDE; + virtual APT::StringView Description_md5() APT_OVERRIDE; virtual unsigned short VersionHash() APT_OVERRIDE; bool LoadReleaseInfo(pkgCache::RlsFileIterator &FileI,FileFd &File, |