You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@mynewt.apache.org by GitBox <gi...@apache.org> on 2021/12/13 14:07:43 UTC

[GitHub] [mynewt-nimble] sjanc commented on pull request #1129: host/gap: fix legacy advertising for BLE_EXT_ADV=1

sjanc commented on pull request #1129:
URL: https://github.com/apache/mynewt-nimble/pull/1129#issuecomment-992515532


   I'm not going to block this if you really prefer auto-enable for missing features, but I'd rather document this behavior for legacy configuration and keep returning error (and maybe provide convenience macros for legacy configurations?).
   
   The thing is that one might not expect that just enabling legacy advertising might result in replying to scan responses (since scanable was not explicitly set)


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@mynewt.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org