rymanluk commented on issue #697: Added extra argument to sync and reset 
collback to simplify building C++ Wrapper
URL: https://github.com/apache/mynewt-nimble/pull/697#issuecomment-566082840
 
 
   @redradist 
   Few cents from my side. 
   I agree with @ccollins476ad  @andrzej-kaczmarek  that backward compatibility 
is important.
   Solution with additional `_extra_` is no a good one as we would end up with 
those `_extra_` API, which is not that great. We need some other clean solution 
which is unknown now.
   In the meantime I think that @andrzej-kaczmarek suggestion for you to keep 
`cb_args` in global `ble_hs_cfg` and use them in your callbacks in the porting 
layer is fair enough.
   Waiting for your contribution. Thanks

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to