[GitHub] [mynewt-nimble] jimmo commented on pull request #998: nimble/porting: Make os_mbuf/os_mpool work on 64-bit.

[GitBox <gi...@apache.org>]

jimmo commented on pull request #998:
URL: https://github.com/apache/mynewt-nimble/pull/998#issuecomment-878271358


   > Yes, lets fix it in core and then resync here.
   > In general I think we should simply start using uintptr where needed (although mynewt was/is 32bit os only).
   
   Thanks -- I'm not too familiar with mynewt (outside of NimBLE). Do you want me to follow this up or is this best left up to the maintainers?
   
   > Could you explain why is this needed? (AFAIR initially it was like that to allow multiple initialization while executing tests)
   
   I'm not too fussed either way, but I noticed that nothing in NimBLE calls os_mempool_module_init and there's no declaration for it in any header files. So this change seems like the safest way to prevent implementors missing it.


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