v01d commented on pull request #385: URL: https://github.com/apache/incubator-nuttx-apps/pull/385#issuecomment-693149779
Thanks, I now understand why the change. I think the idea is reasonable. Maybe someone else wants to take a look and see if there isn't anything critical not considered? BTW, if you don't mind I'd rather we wait for this to be merged https://github.com/apache/incubator-nuttx/pull/1785 since I have that PR ready and the corresponding one for apps. Adapting this PR to that is really simple, but this way we make it in one go. ---------------------------------------------------------------- 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]
