diff options
author | Sybren A. Stüvel <sybren@blender.org> | 2021-09-23 19:22:47 +0300 |
---|---|---|
committer | Sybren A. Stüvel <sybren@blender.org> | 2021-09-23 19:27:19 +0300 |
commit | 18a4dc869d16fca315df32a790843a0068c72a47 (patch) | |
tree | f9ba190f227ad71fb04f7dcf07a99ad1a3269dc8 | |
parent | 502543e46b0c3ccc92db7161b899442f6d7a10f9 (diff) |
Cleanup: UUID, fix clang-tidy warnings
Use explicit `uint32_t` instead of `uint`, add a missing end-of-namespace
comment, and change `auto` to `const auto *`.
No functional changes.
-rw-r--r-- | source/blender/blenlib/BLI_uuid.h | 4 | ||||
-rw-r--r-- | source/blender/blenlib/intern/uuid.cc | 6 |
2 files changed, 5 insertions, 5 deletions
diff --git a/source/blender/blenlib/BLI_uuid.h b/source/blender/blenlib/BLI_uuid.h index 111a3d1eac3..eef37841d15 100644 --- a/source/blender/blenlib/BLI_uuid.h +++ b/source/blender/blenlib/BLI_uuid.h @@ -84,13 +84,13 @@ class bUUID : public ::bUUID { bUUID(const ::bUUID &struct_uuid); /** Initialise from 11 integers, 5 for the regular fields and 6 for the `node` array. */ - bUUID(std::initializer_list<uint> field_values); + bUUID(std::initializer_list<uint32_t> field_values); /** Initialise by parsing the string; undefined behaviour when the string is invalid. */ explicit bUUID(const std::string &string_formatted_uuid); uint64_t hash() const; -}; +}; // namespace blender bool operator==(bUUID uuid1, bUUID uuid2); bool operator!=(bUUID uuid1, bUUID uuid2); diff --git a/source/blender/blenlib/intern/uuid.cc b/source/blender/blenlib/intern/uuid.cc index bc8f67f939c..fe237b8aae6 100644 --- a/source/blender/blenlib/intern/uuid.cc +++ b/source/blender/blenlib/intern/uuid.cc @@ -142,13 +142,13 @@ std::ostream &operator<<(std::ostream &stream, bUUID uuid) namespace blender { -bUUID::bUUID(const std::initializer_list<uint> field_values) +bUUID::bUUID(const std::initializer_list<uint32_t> field_values) { BLI_assert_msg(field_values.size() == 11, "bUUID requires 5 regular fields + 6 `node` values"); - auto field_iter = field_values.begin(); + const auto *field_iter = field_values.begin(); - this->time_low = static_cast<uint32_t>(*field_iter++); + this->time_low = *field_iter++; this->time_mid = static_cast<uint16_t>(*field_iter++); this->time_hi_and_version = static_cast<uint16_t>(*field_iter++); this->clock_seq_hi_and_reserved = static_cast<uint8_t>(*field_iter++); |