Simplify scheduler assignment logic and prevent skipping

src/mbgl/actor/scheduler.cpp

@@ -48,29 +48,22 @@ std::shared_ptr<Scheduler> Scheduler::GetBackground() {
 
 // static
 std::shared_ptr<Scheduler> Scheduler::GetSequenced() {
-    const std::size_t kSchedulersCount = 10;
+    constexpr std::size_t kSchedulersCount = 10;
     static std::vector<std::weak_ptr<Scheduler>> weaks(kSchedulersCount);
     static std::mutex mtx;
     static std::size_t lastUsedIndex = 0u;
 
-    std::lock_guard<std::mutex> lock(mtx);
+    std::lock_guard lock(mtx);
 
-    if (++lastUsedIndex == kSchedulersCount) lastUsedIndex = 0u;
+    lastUsedIndex = (lastUsedIndex + 1) % kSchedulersCount;
 
-    std::shared_ptr<Scheduler> result;
-    for (std::size_t i = 0; i < kSchedulersCount; ++i) {
-        auto& weak = weaks[i];
-        if (auto scheduler = weak.lock()) {
-            if (lastUsedIndex == i) result = scheduler;
-            continue;
-        }
-        result = std::make_shared<SequencedScheduler>();
-        weak = result;
-        lastUsedIndex = i;
-        break;
+    if (auto scheduler = weaks[lastUsedIndex].lock()) {
+        return scheduler;
+    } else {
+        auto result = std::make_shared<SequencedScheduler>();
+        weaks[lastUsedIndex] = result;
+        return result;
     }
-
-    return result;
 }
 
 } // namespace mbgl