Review fixes

laralem · Nov 30, 2015 · 8505cb9 · 8505cb9
1 parent 0ef3327
commit 8505cb9
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/drape_frontend/message_queue.cpp b/drape_frontend/message_queue.cpp
@@ -37,12 +37,12 @@ void MessageQueue::PushMessage(dp::TransferPointer<Message> message, MessagePrio
   {
   case MessagePriority::Normal:
     {
-      m_messages.push_back(dp::MasterPointer<Message>(message));
+      m_messages.emplace_back(message);
       break;
     }
   case MessagePriority::High:
     {
-      m_messages.insert(m_messages.begin(), dp::MasterPointer<Message>(message));
+      m_messages.emplace_front(message);
       break;
     }
   default:

diff --git a/map/framework.cpp b/map/framework.cpp
@@ -897,6 +897,8 @@ void Framework::EnterBackground()
 #ifndef OMIM_OS_ANDROID
   ClearAllCaches();
 #endif
+
+  ASSERT(!m_drapeEngine.IsNull(), ("Drape engine has not been initialized yet"));
   if (!m_drapeEngine.IsNull())
     m_drapeEngine->SetRenderingEnabled(false);
 }
@@ -905,6 +907,7 @@ void Framework::EnterForeground()
 {
   m_startForegroundTime = my::Timer::LocalTime();
 
+  ASSERT(!m_drapeEngine.IsNull(), ("Drape engine has not been initialized yet"));
   if (!m_drapeEngine.IsNull())
     m_drapeEngine->SetRenderingEnabled(true);
 }