You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Instead of having two functions set_*_callback and unset_*_callback for each callback, I think it would be simpler to only have set_*_callback, and have it take an optional function as argument (the Some case installing a callback, the None cas removing it). What do you think?
The text was updated successfully, but these errors were encountered:
Instead of having two functions set_*_callback and unset_*_callback for each
callback, I think it would be simpler to only have set_*_callback, and have it
take an optional function as argument (the Some case installing a callback, the
None cas removing it). What do you think?
I find a bit bizarre to have the default with no extra arg removing the call-back.
I think a bit and look at that. I did not forget your idea to use a macro on
the C size by the way.
Instead of having two functions
set_*_callback
andunset_*_callback
for each callback, I think it would be simpler to only haveset_*_callback
, and have it take an optional function as argument (theSome
case installing a callback, theNone
cas removing it). What do you think?The text was updated successfully, but these errors were encountered: