v01d commented on pull request #1248: URL: https://github.com/apache/incubator-nuttx/pull/1248#issuecomment-646875326
Ok, I can make it simply return an error if period == 0. Adding a min_delay seems redundant when the oneshot timer already receives a "resolution" parameter. It would be expected that this is in particular the minimum number allowed. Should this be a recoverable error? Or a DEBUGASSERT() just like the `DEBUGASSERT(period <= UINT32_MAX);`? ---------------------------------------------------------------- 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: us...@infra.apache.org
