diff --git a/map/bookmark.hpp b/map/bookmark.hpp
index a43675e195..d31e557da4 100644
--- a/map/bookmark.hpp
+++ b/map/bookmark.hpp
@@ -89,7 +89,7 @@ public:
   void SetData(BookmarkData const & data) { m_data = data; }
   BookmarkData const & GetData() const { return m_data; }
 
-  virtual Type GetMarkType() const { return UserMark::Type::BOOKMARK; }
+  virtual Type GetMarkType() const override { return UserMark::Type::BOOKMARK; }
   virtual void FillLogEvent(TEventContainer & details) const override;
 
   string const & GetName() const { return m_data.GetName(); }
diff --git a/map/map.pro b/map/map.pro
index f78f9bc193..9405e92e1f 100644
--- a/map/map.pro
+++ b/map/map.pro
@@ -117,7 +117,6 @@ SOURCES += \
     track.cpp \
     alfa_animation_task.cpp \
     user_mark_container.cpp \
-    user_mark.cpp \
     user_mark_dl_cache.cpp \
     anim_phase_chain.cpp \
     pin_click_manager.cpp \
diff --git a/map/user_mark.cpp b/map/user_mark.cpp
deleted file mode 100644
index f8b128456c..0000000000
--- a/map/user_mark.cpp
+++ /dev/null
@@ -1,28 +0,0 @@
-#include "user_mark.hpp"
-#include "user_mark_container.hpp"
-
-#include "../indexer/mercator.hpp"
-
-UserMark::UserMark(m2::PointD const & ptOrg, UserMarkContainer * container)
-  : m_ptOrg(ptOrg)
-  , m_container(container)
-{
-}
-
-UserMark::~UserMark() {}
-
-UserMarkContainer const * UserMark::GetContainer() const
-{
-  return m_container;
-}
-
-m2::PointD const & UserMark::GetOrg() const
-{
-  return m_ptOrg;
-}
-
-void UserMark::GetLatLon(double & lat, double & lon) const
-{
-  lon = MercatorBounds::XToLon(m_ptOrg.x);
-  lat = MercatorBounds::YToLat(m_ptOrg.y);
-}
diff --git a/map/user_mark.hpp b/map/user_mark.hpp
index 7b0ee3e277..ca7307080d 100644
--- a/map/user_mark.hpp
+++ b/map/user_mark.hpp
@@ -6,7 +6,6 @@
 
 #include "../search/result.hpp"
 
-#include "../std/noncopyable.hpp"
 #include "../std/string.hpp"
 #include "../std/unique_ptr.hpp"
 
@@ -21,8 +20,11 @@ namespace graphics
 
 class UserMarkCopy;
 
-class UserMark : private noncopyable
+class UserMark
 {
+  UserMark(UserMark const &) = delete;
+  UserMark& operator=(UserMark const &) = delete;
+
 public:
   enum class Type
   {
@@ -33,12 +35,20 @@ public:
     MY_POSITION
   };
 
-  UserMark(m2::PointD const & ptOrg, UserMarkContainer * container);
-  virtual ~UserMark();
+  UserMark(m2::PointD const & ptOrg, UserMarkContainer * container)
+    : m_ptOrg(ptOrg), m_container(container)
+  {
+  }
 
-  UserMarkContainer const * GetContainer() const;
-  m2::PointD const & GetOrg() const;
-  void GetLatLon(double & lat, double & lon) const;
+  virtual ~UserMark() {}
+
+  UserMarkContainer const * GetContainer() const { return m_container; }
+  m2::PointD const & GetOrg() const { return m_ptOrg; }
+  void GetLatLon(double & lat, double & lon) const
+  {
+    lon = MercatorBounds::XToLon(m_ptOrg.x);
+    lat = MercatorBounds::YToLat(m_ptOrg.y);
+  }
   virtual bool IsCustomDrawable() const { return false;}
   virtual Type GetMarkType() const = 0;
   virtual unique_ptr<UserMarkCopy> Copy() const = 0;
@@ -97,7 +107,7 @@ public:
   {
   }
 
-  UserMark::Type GetMarkType() const { return UserMark::Type::API; }
+  UserMark::Type GetMarkType() const override { return UserMark::Type::API; }
 
   string const & GetName() const { return m_name; }
   void SetName(string const & name) { m_name = name; }
@@ -139,7 +149,7 @@ public:
   {
   }
 
-  UserMark::Type GetMarkType() const { return UserMark::Type::SEARCH; }
+  UserMark::Type GetMarkType() const override { return UserMark::Type::SEARCH; }
 
   search::AddressInfo const & GetInfo() const { return m_info; }
   void SetInfo(search::AddressInfo const & info) { m_info = info; }
@@ -173,7 +183,7 @@ public:
   PoiMarkPoint(UserMarkContainer * container)
     : SearchMarkPoint(m2::PointD(0.0, 0.0), container) {}
 
-  UserMark::Type GetMarkType() const { return UserMark::Type::POI; }
+  UserMark::Type GetMarkType() const override { return UserMark::Type::POI; }
   unique_ptr<UserMarkCopy> Copy() const override
   {
     return unique_ptr<UserMarkCopy>(new UserMarkCopy(this, false));
@@ -190,12 +200,10 @@ public:
 
 class MyPositionMarkPoint : public PoiMarkPoint
 {
-  typedef PoiMarkPoint base_t;
 public:
-  MyPositionMarkPoint(UserMarkContainer * container)
-    : base_t(container) {}
+  MyPositionMarkPoint(UserMarkContainer * container) : PoiMarkPoint(container) {}
 
-  UserMark::Type GetMarkType() const { return UserMark::Type::MY_POSITION; }
+  UserMark::Type GetMarkType() const override { return UserMark::Type::MY_POSITION; }
   virtual void FillLogEvent(TEventContainer & details) const override
   {
     PoiMarkPoint::FillLogEvent(details);