CoreContext::Snoop is named backwards #469
Comments
|
Agreed, this has been a long standing issue. I will rename this method to This method is extremely widely used, deprecation will take a month or two. |
codemercenary
added a commit
that referenced
this issue
Mar 30, 2015
Fixes #469. The Snoop method is misleading. We cannot rename this method because it is too extensively used, but we can start a deprecation policy and get it fixed eventually.
codemercenary
added a commit
that referenced
this issue
Apr 2, 2015
Fixes #469. The Snoop method is misleading. We cannot rename this method because it is too extensively used, but we can start a deprecation policy and get it fixed eventually.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
E.g. if c is a context and s is an instance of a class with an AutoFilter method, then the call
c->Snoop(s)
looks like it should cause c to snoop s, but it really means
s will snoop c. This is misleading.The text was updated successfully, but these errors were encountered: