On Tue May 23 16:24:16 2023 +0000, Connor McAdams wrote:
I'd generally agree, but the interface is called `IRawElementProviderAdviseEvents`, and the methods are called `AdviseEventAdded` and `AdviseEventRemoved`. IMO, introducing the word 'notify' could add confusion, given someone might start looking for 'notify' or 'notifier' in the code. I'm a bit split on that. I definitely like the word notify better than advise, but it does introduce a split between the interface name/method names and our code potentially.
Also, the base word here is `advise` and not `advice`, not sure if that makes a difference :)
Edit: Actually duh, advise just means to give advice.