Change all of the mentions of 'query' in NotificationToken to 'notifier'
This commit is contained in:
parent
31c3982bff
commit
fe5564f40e
|
@ -23,21 +23,21 @@
|
||||||
using namespace realm;
|
using namespace realm;
|
||||||
using namespace realm::_impl;
|
using namespace realm::_impl;
|
||||||
|
|
||||||
NotificationToken::NotificationToken(std::shared_ptr<_impl::BackgroundCollection> query, size_t token)
|
NotificationToken::NotificationToken(std::shared_ptr<_impl::BackgroundCollection> notifier, size_t token)
|
||||||
: m_query(std::move(query)), m_token(token)
|
: m_notifier(std::move(notifier)), m_token(token)
|
||||||
{
|
{
|
||||||
}
|
}
|
||||||
|
|
||||||
NotificationToken::~NotificationToken()
|
NotificationToken::~NotificationToken()
|
||||||
{
|
{
|
||||||
// m_query itself (and not just the pointed-to thing) needs to be accessed
|
// m_notifier itself (and not just the pointed-to thing) needs to be accessed
|
||||||
// atomically to ensure that there are no data races when the token is
|
// atomically to ensure that there are no data races when the token is
|
||||||
// destroyed after being modified on a different thread.
|
// destroyed after being modified on a different thread.
|
||||||
// This is needed despite the token not being thread-safe in general as
|
// This is needed despite the token not being thread-safe in general as
|
||||||
// users find it very surpringing for obj-c objects to care about what
|
// users find it very surpringing for obj-c objects to care about what
|
||||||
// thread they are deallocated on.
|
// thread they are deallocated on.
|
||||||
if (auto query = m_query.exchange({})) {
|
if (auto notifier = m_notifier.exchange({})) {
|
||||||
query->remove_callback(m_token);
|
notifier->remove_callback(m_token);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -46,10 +46,10 @@ NotificationToken::NotificationToken(NotificationToken&& rgt) = default;
|
||||||
NotificationToken& NotificationToken::operator=(realm::NotificationToken&& rgt)
|
NotificationToken& NotificationToken::operator=(realm::NotificationToken&& rgt)
|
||||||
{
|
{
|
||||||
if (this != &rgt) {
|
if (this != &rgt) {
|
||||||
if (auto query = m_query.exchange({})) {
|
if (auto notifier = m_notifier.exchange({})) {
|
||||||
query->remove_callback(m_token);
|
notifier->remove_callback(m_token);
|
||||||
}
|
}
|
||||||
m_query = std::move(rgt.m_query);
|
m_notifier = std::move(rgt.m_notifier);
|
||||||
m_token = rgt.m_token;
|
m_token = rgt.m_token;
|
||||||
}
|
}
|
||||||
return *this;
|
return *this;
|
||||||
|
|
|
@ -35,7 +35,7 @@ namespace _impl {
|
||||||
// A token which keeps an asynchronous query alive
|
// A token which keeps an asynchronous query alive
|
||||||
struct NotificationToken {
|
struct NotificationToken {
|
||||||
NotificationToken() = default;
|
NotificationToken() = default;
|
||||||
NotificationToken(std::shared_ptr<_impl::BackgroundCollection> query, size_t token);
|
NotificationToken(std::shared_ptr<_impl::BackgroundCollection> notifier, size_t token);
|
||||||
~NotificationToken();
|
~NotificationToken();
|
||||||
|
|
||||||
NotificationToken(NotificationToken&&);
|
NotificationToken(NotificationToken&&);
|
||||||
|
@ -45,7 +45,7 @@ struct NotificationToken {
|
||||||
NotificationToken& operator=(NotificationToken const&) = delete;
|
NotificationToken& operator=(NotificationToken const&) = delete;
|
||||||
|
|
||||||
private:
|
private:
|
||||||
util::AtomicSharedPtr<_impl::BackgroundCollection> m_query;
|
util::AtomicSharedPtr<_impl::BackgroundCollection> m_notifier;
|
||||||
size_t m_token;
|
size_t m_token;
|
||||||
};
|
};
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue