From 5864abe6c82acc9c85607864fbbc9b873392162e Mon Sep 17 00:00:00 2001 From: Arnaud Giersch Date: Thu, 4 Apr 2019 14:44:13 +0200 Subject: [PATCH 1/1] Cosmetic cleanups. --- include/xbt/Extendable.hpp | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/include/xbt/Extendable.hpp b/include/xbt/Extendable.hpp index 37a7d41c59..37946b6958 100644 --- a/include/xbt/Extendable.hpp +++ b/include/xbt/Extendable.hpp @@ -52,9 +52,8 @@ private: public: static size_t extension_create(void (*deleter)(void*)) { - std::size_t res = deleters_.size(); deleters_.push_back(deleter); - return res; + return deleters_.size() - 1; } template static Extension extension_create(void (*deleter)(void*)) @@ -85,17 +84,14 @@ public: // Type-unsafe versions of the facet access methods: void* extension(std::size_t rank) const { - if (rank >= extensions_.size()) - return nullptr; - else - return extensions_.at(rank); + return rank < extensions_.size() ? extensions_[rank] : nullptr; } void extension_set(std::size_t rank, void* value, bool use_dtor = true) { if (rank >= extensions_.size()) extensions_.resize(rank + 1, nullptr); void* old_value = this->extension(rank); - extensions_.at(rank) = value; + extensions_[rank] = value; if (use_dtor && old_value != nullptr && deleters_[rank]) deleters_[rank](old_value); } -- 2.20.1