GpgFrontend/gpgme
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

cpp: Fix version info comparison

Browse Source 
* lang/cpp/src/engineinfo.h (EngineInfo::Version::operator<):
Fix logic.
* lang/cpp/src/engineinfo.h (EngineInfo::Version::operator>):
New.
* NEWS: Mention added API

--
This fixes a logic error that 2.2.0 < 2.1.19 would return true.
This commit is contained in:
Andre Heinecke 2017-09-04 11:23:56 +02:00
parent 47f61df070
commit 58d7bcead3
No known key found for this signature in database
GPG Key ID: 2978E9D40CBABA5C
2 changed files with 19 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
NEWS
View File

@ -14,6 +14,7 @@ Noteworthy changes in version 1.10.0 (unreleased)
gpgme_set_ctx_flag EXTENDED: New flag 'auto-key-retrieve'. gpgme_set_ctx_flag EXTENDED: New flag 'auto-key-retrieve'.
cpp: DecryptionResult::isDeVs NEW. cpp: DecryptionResult::isDeVs NEW.
cpp: Signature::isDeVs NEW. cpp: Signature::isDeVs NEW.
cpp: EngineInfo::Version::operator> NEW.
py: DecryptResult EXTENDED: New boolean field 'is_de_vs'. py: DecryptResult EXTENDED: New boolean field 'is_de_vs'.
py: Signature EXTENDED: New boolean field 'is_de_vs'. py: Signature EXTENDED: New boolean field 'is_de_vs'.
py: GpgError EXTENDED: Partial results in 'results'. py: GpgError EXTENDED: Partial results in 'results'.

21
lang/cpp/src/engineinfo.h
View File

@ -71,20 +71,29 @@ public:
bool operator < (const Version& other) bool operator < (const Version& other)
{ {
if (major < other.major) if (major > other.major ||
return true; (major == other.major && minor > other.minor) ||
if (minor < other.minor) (major == other.major && minor == other.minor && patch > other.patch) ||
return true; (major >= other.major && minor >= other.minor && patch >= other.patch)) {
if (patch < other.patch)
return true;
return false; return false;
} }
return true;
}
bool operator < (const char* other) bool operator < (const char* other)
{ {
return operator<(Version(other)); return operator<(Version(other));
} }
bool operator > (const char* other)
{
return !operator<(Version(other));
}
bool operator > (const Version & other)
{
return !operator<(other);
}
bool operator == (const Version& other) bool operator == (const Version& other)
{ {
return major == other.major return major == other.major