gian/ceph-mirror
1
0
Fork 0
mirror of https://github.com/ceph/ceph synced 2026-08-02 07:03:18 +00:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

osd/SnapMapper: use fmt::format() instead of fmt::sprintf()

Browse Source 
fmt::sprintf() is a legacy API and cannot use custom formatters.

Signed-off-by: Max Kellermann <max.kellermann@ionos.com>
This commit is contained in:
Max Kellermann 2024-10-14 22:29:39 +02:00
parent fba2a4e5c5
commit adfbcec205
1 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/osd/SnapMapper.cc
View File

@ -15,7 +15,6 @@
#include "SnapMapper.h"
#include <fmt/printf.h>
#include <fmt/ranges.h>
#include "common/ceph_context.h"
@ -279,7 +278,7 @@ string SnapMapper::get_prefix(int64_t pool, snapid_t snap)
// note: the snap_id is to be formatted as a 64-bit hex number,
// and not according to the text representation of snapid_t
ceph_assert(snap != CEPH_NOSNAP && snap != CEPH_SNAPDIR);
return fmt::sprintf("%s%lld_%.16X_",
return fmt::format("{}{}_{:016X}_",
MAPPING_PREFIX,
pool,
static_cast<uint64_t>(snap));
@ -873,7 +872,7 @@ void SnapMapper::update_snap_map(
string SnapMapper::make_purged_snap_key(int64_t pool, snapid_t last)
{
ceph_assert(last != CEPH_NOSNAP && last != CEPH_SNAPDIR);
return fmt::sprintf("%s_%lld_%016llx",
return fmt::format("{}_{}_{:016x}",
PURGED_SNAP_PREFIX,
pool,
static_cast<uint64_t>(last));