dphfox
Add state handlers for OnChange, OnEvent, etc.
Fixes #109 This will allow for state objects to be used for Change and Event callbacks.
Is this designed enough yet?
The original issue mentions the gain to be "avoiding unnecessary memory from an unused connection", but this implementation does not accomplish that.
This implementation also breaks when the state result is nil, as was given as a usecase in the original issue.
In general, the concept of using a state to switch between different callbacks does not seem so intuitive to me. Especially with no performance/memory gain, code readability would be improved by just using a regular callback that performs the logic when the event is called.
This implementation also breaks when the state result is nil, as was given as a usecase in the original issue.
Didn't realise this! Will update!
"avoiding unnecessary memory from an unused connection", but this implementation does not accomplish that.
Mind elaborating? My mind is a bit off today haha
Closing this PR due to a lack of motion. I'll be happy to merge a more up-to-date variant.
