From d0367de056d3aa4c9c9f3c4d19f4233899a33f0d Mon Sep 17 00:00:00 2001 From: Ingo Weinhold Date: Thu, 18 Apr 2013 14:38:24 +0200 Subject: [PATCH] packagefs: Switch to new rule for pre-release version part --- src/add-ons/kernel/file_systems/packagefs/Version.cpp | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/add-ons/kernel/file_systems/packagefs/Version.cpp b/src/add-ons/kernel/file_systems/packagefs/Version.cpp index b5206bb5e2..3ea5660d52 100644 --- a/src/add-ons/kernel/file_systems/packagefs/Version.cpp +++ b/src/add-ons/kernel/file_systems/packagefs/Version.cpp @@ -173,7 +173,9 @@ Version::ToString(char* buffer, size_t bufferSize) const // We need to normalize the version string somewhat. If a subpart is given, // make sure that also the superparts are defined, using a placeholder. This // avoids clashes, e.g. if one version defines major and minor and one only - // major and micro. + // major and micro. In principle that should not be necessary, though. Valid + // packages should have valid versions, which means that the existence of a + // subpart implies the existence of all superparts. const char* major = fMajor; const char* minor = fMinor; const char* micro = fMicro; @@ -197,7 +199,7 @@ Version::ToString(char* buffer, size_t bufferSize) const if (fPreRelease != NULL) { size_t offset = std::min(bufferSize, size); - size += snprintf(buffer + offset, bufferSize - offset, "-%s", + size += snprintf(buffer + offset, bufferSize - offset, "[%s]", fPreRelease); }