Michele Caini 8 years ago
parent
commit
631bf42f84

+ 2 - 2
src/entt/entity/registry.hpp

@@ -947,7 +947,7 @@ public:
     template<typename Func>
     void each(Func func) const {
         if(available) {
-            for(auto pos = entities.size(); pos > size_type{0}; --pos) {
+            for(auto pos = entities.size(); pos; --pos) {
                 const entity_type curr = pos - 1;
                 const auto entt = entities[curr] & traits_type::entity_mask;
 
@@ -956,7 +956,7 @@ public:
                 }
             }
         } else {
-            for(auto pos = entities.size(); pos > size_type{0}; --pos) {
+            for(auto pos = entities.size(); pos; --pos) {
                 func(entities[pos-1]);
             }
         }

+ 2 - 2
src/entt/entity/sparse_set.hpp

@@ -379,7 +379,7 @@ public:
 
         pos_type pos = direct.size() - 1;
 
-        while(pos > 0 && from != to) {
+        while(pos && from != to) {
             if(has(*from)) {
                 if(*from != direct[pos]) {
                     swap(pos, get(*from));
@@ -692,7 +692,7 @@ public:
         pos_type pos = underlying_type::size() - 1;
         const auto *local = underlying_type::data();
 
-        while(pos > 0 && from != to) {
+        while(pos && from != to) {
             const auto curr = *from;
 
             if(underlying_type::has(curr)) {

+ 1 - 1
src/entt/process/scheduler.hpp

@@ -274,7 +274,7 @@ public:
     void update(Delta delta, void *data = nullptr) {
         bool clean = false;
 
-        for(auto pos = handlers.size(); pos > 0; --pos) {
+        for(auto pos = handlers.size(); pos; --pos) {
             auto &handler = handlers[pos-1];
             const bool dead = handler.update(handler, delta, data);
             clean = clean || dead;

+ 1 - 1
src/entt/signal/dispatcher.hpp

@@ -181,7 +181,7 @@ public:
         const auto buf = buffer(mode);
         mode = !mode;
 
-        for(auto pos = wrappers.size(); pos > decltype(pos){0}; --pos) {
+        for(auto pos = wrappers.size(); pos; --pos) {
             auto &wrapper = wrappers[pos-1];
 
             if(wrapper) {

+ 2 - 2
src/entt/signal/sigh.hpp

@@ -229,7 +229,7 @@ public:
      * @param args Arguments to use to invoke listeners.
      */
     void publish(Args... args) {
-        for(auto pos = calls.size(); pos > size_type{0}; --pos) {
+        for(auto pos = calls.size(); pos; --pos) {
             auto &call = calls[pos-1];
             call.second(call.first, args...);
         }
@@ -243,7 +243,7 @@ public:
     collector_type collect(Args... args) {
         collector_type collector;
 
-        for(auto pos = calls.size(); pos > size_type{0}; --pos) {
+        for(auto pos = calls.size(); pos; --pos) {
             auto &call = calls[pos-1];
 
             if(!this->invoke(collector, call.second, call.first, args...)) {

+ 1 - 1
src/entt/signal/signal.hpp

@@ -169,7 +169,7 @@ public:
     void publish(Args... args) {
         std::vector<call_type> next;
 
-        for(auto pos = calls.size(); pos > size_type{0}; --pos) {
+        for(auto pos = calls.size(); pos; --pos) {
             auto &call = calls[pos-1];
 
             if((call.second)(call.first, args...)) {