MapContext::updated doesn't need to be atomic

2 files changed, 9 insertions, 9 deletions

diff --git a/src/mbgl/map/map_context.cpp b/src/mbgl/map/map_context.cpp
index e10a46e99b..da71b787df 100644
--- a/src/mbgl/map/map_context.cpp
+++ b/src/mbgl/map/map_context.cpp
@@ -188,27 +188,25 @@ void MapContext::update() {
     const auto now = Clock::now();
     data.setAnimationTime(now);
 
-    auto u = updated.exchange(static_cast<UpdateType>(Update::Nothing)) |
-             data.transform.updateTransitions(now);
-
+    updated |= data.transform.updateTransitions(now);
     transformState = data.transform.currentState();
 
-    if (u & static_cast<UpdateType>(Update::Debug)) {
+    if (updated & static_cast<UpdateType>(Update::Debug)) {
         assert(painter);
         painter->setDebug(data.getDebug());
     }
 
     if (style) {
-        if (u & static_cast<UpdateType>(Update::DefaultTransitionDuration)) {
+        if (updated & static_cast<UpdateType>(Update::DefaultTransitionDuration)) {
             style->setDefaultTransitionDuration(data.getDefaultTransitionDuration());
         }
 
-        if (u & static_cast<UpdateType>(Update::Classes)) {
+        if (updated & static_cast<UpdateType>(Update::Classes)) {
             style->cascade(data.getClasses());
         }
 
-        if (u & static_cast<UpdateType>(Update::Classes) ||
-            u & static_cast<UpdateType>(Update::Zoom)) {
+        if (updated & static_cast<UpdateType>(Update::Classes) ||
+            updated & static_cast<UpdateType>(Update::Zoom)) {
             style->recalculate(transformState.getNormalizedZoom(), now);
         }
 
@@ -220,6 +218,8 @@ void MapContext::update() {
 
         view.invalidate([this] { render(); });
     }
+
+    updated = static_cast<UpdateType>(Update::Nothing);
 }
 
 void MapContext::renderStill(StillImageCallback fn) {
diff --git a/src/mbgl/map/map_context.hpp b/src/mbgl/map/map_context.hpp
index 6e7b082f2a..2c9f287086 100644
--- a/src/mbgl/map/map_context.hpp
+++ b/src/mbgl/map/map_context.hpp
@@ -74,7 +74,7 @@ private:
     Environment env;
     EnvironmentScope envScope;
 
-    std::atomic<UpdateType> updated { static_cast<UpdateType>(Update::Nothing) };
+    UpdateType updated { static_cast<UpdateType>(Update::Nothing) };
     std::unique_ptr<uv::async> asyncUpdate;
 
     std::unique_ptr<Worker> workers;