Move camera and time serialization and deserialization to cpp-files

include/openspace/util/camera.h

@@ -29,7 +29,6 @@
 
 // open space includes
 #include <openspace/util/powerscaledcoordinate.h>
-#include <openspace/util/syncbuffer.h>
 #include <openspace/rendering/renderengine.h>
 
 // glm includes
@@ -181,16 +180,8 @@ namespace openspace {
 
     private:
         struct SyncData {
-            void serialize(SyncBuffer* syncBuffer) { 
-                syncBuffer->encode(position); 
-                syncBuffer->encode(rotation);
-                syncBuffer->encode(scaling);
-            }
-            void deserialize(SyncBuffer* syncBuffer) { 
-                syncBuffer->decode(position);
-                syncBuffer->decode(rotation);
-                syncBuffer->decode(scaling);
-            }
+            void serialize(SyncBuffer* syncBuffer);
+            void deserialize(SyncBuffer* syncBuffer);
             
             Vec3 position;
             Quat rotation;

include/openspace/util/time.h

@@ -26,7 +26,6 @@
 #define __TIME_H__
 
 #include <openspace/scripting/scriptengine.h>
-#include <openspace/util/syncbuffer.h>
 #include <mutex>
 #include <string>
 
@@ -209,17 +208,9 @@ private:
     static Time* _instance; ///< The singleton instance
 
     struct SyncData {
-        void serialize(SyncBuffer* syncBuffer) { 
-            syncBuffer->encode(time);
-            syncBuffer->encode(dt);
-            syncBuffer->encode(timeJumped);
-        };
+        void serialize(SyncBuffer* syncBuffer);
 
-        void deserialize(SyncBuffer* syncBuffer) { 
-            syncBuffer->decode(time);
-            syncBuffer->decode(dt);
-            syncBuffer->decode(timeJumped);
-        }
+        void deserialize(SyncBuffer* syncBuffer);
         
         double time = -1.0;
         double dt = 1.0;

src/util/camera.cpp

@@ -299,6 +299,18 @@ namespace openspace {
     }
 
     void Camera::updateDoubleBuffer(){
+        std::lock_guard<std::mutex> _lock(_mutex);
         local = synced;
     }
+
+    void Camera::SyncData::serialize(SyncBuffer* syncBuffer) {
+        syncBuffer->encode(position);
+        syncBuffer->encode(rotation);
+        syncBuffer->encode(scaling);
+    }
+    void Camera::SyncData::deserialize(SyncBuffer* syncBuffer) {
+        syncBuffer->decode(position);
+        syncBuffer->decode(rotation);
+        syncBuffer->decode(scaling);
+    }
 } // namespace openspace

src/util/time.cpp

@@ -174,7 +174,21 @@ bool Time::paused() const {
     return _timePaused;
 }
 void Time::updateDoubleBuffer() {
+    _syncMutex.lock();
     local = synced;
+    _syncMutex.unlock();
+}
+
+void Time::SyncData::serialize(SyncBuffer* syncBuffer) {
+    syncBuffer->encode(time);
+    syncBuffer->encode(dt);
+    syncBuffer->encode(timeJumped);
+};
+
+void Time::SyncData::deserialize(SyncBuffer* syncBuffer) {
+    syncBuffer->decode(time);
+    syncBuffer->decode(dt);
+    syncBuffer->decode(timeJumped);
 }