linguini1 commented on PR #18430: URL: https://github.com/apache/nuttx/pull/18430#issuecomment-3952879651
> @linguini1 I'd suggest to mark all legacy sensor drivers as deprecated - I'm not sure how that's done (or could be done) in NuttX however? snerr says "driver is deprecated" inside register function? GCC warning pragma? > > When I need support for a new device, my first instinct is to look for something similar in the codebase - as I expect that code similar to already merged code will be accepted more easily. Thank you @jeanthom , I'll open an issue to do that (I think a GCC warning pragma + comment might be good for those reading + compiling code). Good suggestion! I always feel bad when this happens because truthfully the documentation isn't particularly navigable and so many contributors will never encounter these "best practices" there to begin with. Sorry! -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
