search

ARMmbed / nrf51-sdk

Module to contain files provided by the nordic nRF51 SDK
Other
13 stars 17 forks source link

Add files from peer manager from nRF51 SDK 10.0.0 #15

Closed andresag01 closed 8 years ago

andresag01 commented 8 years ago

Add a subset of the files from the peer manager from nRF51 SDK 10.0.0. This code is required to implement whitelising in the ble-nrf51822 module.

@pan- @LiyouZhou

andresag01 commented 8 years ago

NOTE: please try to build something before merging.

NOTE: Some of the files I have included are just headers and I left out the actual implementation since we actually do not require this. Would you like me to include the implementation as well for completeness?

LiyouZhou commented 8 years ago

Apart from the indentation, I'm happy with this. Tested ble heartrate example, it compiles fine. +1

andresag01 commented 8 years ago

@LiyouZhou I will fix the indentation and also include some extra files such as the actual implementation of the headers I already added. Even though it is not required, I think it would be good to include the implementation for completeness.

pan- commented 8 years ago

I'm fine with this PR, I will merge it when the indentation is fixed.

andresag01 commented 8 years ago

@pan- @LiyouZhou Fixed indentation. Just a note, that file is completely written using tabs rather than spaces, we should change that...