Killing a few gcc warnings (#88)

* killing a few gcc warnings

src/collection_notifications.cpp

@@ -41,7 +41,7 @@ NotificationToken::~NotificationToken()
     }
 }
 
-NotificationToken::NotificationToken(NotificationToken&& rgt) = default;
+NotificationToken::NotificationToken(NotificationToken&&) = default;
 
 NotificationToken& NotificationToken::operator=(realm::NotificationToken&& rgt)
 {

src/results.cpp

@@ -87,7 +87,7 @@ Results::Results(SharedRealm r, const ObjectSchema& o, TableView tv, SortOrder s
     REALM_ASSERT(m_sort.column_indices.size() == m_sort.ascending.size());
 }
 
-Results::Results(const Results& other) = default;
+Results::Results(const Results&) = default;
 
 // Cannot be defaulted as TableViewBase::operator= is missing from the core static library.
 // Delegate to the copy constructor and move-assignment operators instead.
@@ -529,6 +529,7 @@ bool Results::is_in_table_order() const
         case Mode::TableView:
             return m_table_view.is_in_table_order();
     }
+    REALM_UNREACHABLE(); // keep gcc happy
 }
 
 void Results::Internal::set_table_view(Results& results, realm::TableView &&tv)