diff --git a/base/base_tests/threaded_list_test.cpp b/base/base_tests/threaded_list_test.cpp
index 48de443bce..6b8ca015c0 100644
--- a/base/base_tests/threaded_list_test.cpp
+++ b/base/base_tests/threaded_list_test.cpp
@@ -11,7 +11,6 @@
 struct ThreadedListProcessor : public threads::IRoutine
 {
   ThreadedList<int> * m_p;
-  int m_data;
   list<int> * m_res;
   int m_id;
 
@@ -35,7 +34,6 @@ struct ThreadedListProcessor : public threads::IRoutine
 struct ThreadedPriorityQueueProcessor : public threads::IRoutine
 {
   ThreadedPriorityQueue<int> * m_p;
-  int m_data;
   list<int> * m_res;
   int m_id;
 
diff --git a/base/commands_queue.cpp b/base/commands_queue.cpp
index 76d6bf4dd0..8595fbfa0c 100644
--- a/base/commands_queue.cpp
+++ b/base/commands_queue.cpp
@@ -38,9 +38,6 @@ namespace core
     }
   }
 
-  CommandsQueue::Routine::Routine() : m_env(-1)
-  {}
-
   CommandsQueue::Routine::Routine(CommandsQueue * parent, int idx)
     : m_parent(parent), m_idx(idx), m_env(idx)
   {}
@@ -108,14 +105,15 @@ namespace core
   }
 
   CommandsQueue::CommandsQueue(size_t executorsCount)
-    : m_executorsCount(executorsCount), m_cmdId(0), m_activeCommands(0)
+    : m_executors(new Executor[executorsCount]), m_executorsCount(executorsCount),
+      m_cmdId(0), m_activeCommands(0)
   {
-    m_executors = new Executor[executorsCount];
-    m_executorsCount = executorsCount;
   }
 
   CommandsQueue::~CommandsQueue()
-  {}
+  {
+    // @TODO memory leak in m_executors? call Cancel()?
+  }
 
   void CommandsQueue::Cancel()
   {
diff --git a/base/commands_queue.hpp b/base/commands_queue.hpp
index 7245a20658..d293d6d6aa 100644
--- a/base/commands_queue.hpp
+++ b/base/commands_queue.hpp
@@ -102,7 +102,6 @@ namespace core
 
     public:
 
-      Routine();
       Routine(CommandsQueue * parent, int idx);
 
       void Do();
diff --git a/coding/reader.hpp b/coding/reader.hpp
index e69c636371..bb4e4ed08f 100644
--- a/coding/reader.hpp
+++ b/coding/reader.hpp
@@ -35,8 +35,6 @@ public:
 class MemReader : public Reader
 {
 public:
-  MemReader() {} // TODO: Remove MemReader().
-
   // Construct from block of memory.
   MemReader(void const * pData, size_t size)
     : m_pData(static_cast<char const *>(pData)), m_size(size)
diff --git a/coding/trie_builder.hpp b/coding/trie_builder.hpp
index f96f9fd43a..8e72164c7f 100644
--- a/coding/trie_builder.hpp
+++ b/coding/trie_builder.hpp
@@ -124,7 +124,6 @@ struct ChildInfo
   typedef buffer_vector<uint8_t, 8> EdgeValueStorageType;
   EdgeValueStorageType m_edgeValue;
 
-  ChildInfo() {}
   ChildInfo(bool isLeaf, uint32_t size, TrieChar c) : m_isLeaf(isLeaf), m_size(size), m_edge(1, c)
   {
   }
diff --git a/geometry/packer.cpp b/geometry/packer.cpp
index 074d9fa6d1..bf90e3f6aa 100644
--- a/geometry/packer.cpp
+++ b/geometry/packer.cpp
@@ -8,6 +8,7 @@ namespace m2
   Packer::Packer()
     : m_currentX(0),
     m_currentY(0),
+    m_yStep(0),
     m_width(0),
     m_height(0),
     m_currentHandle(0),