anchao commented on PR #15959: URL: https://github.com/apache/nuttx/pull/15959#issuecomment-2724655488
> > @xiaoxiang781216 So you expect developers to write code like this? > > > > > > ``` > > > struct wdog_periods_s pwdog; > > > struct wdog_s wdog; > > > int ret; > > > > > > if (...) > > > ret = wd_start(&wdog, ...); > > > else > > > ret = wd_start_period(&pwdog, ...); > > > ``` > > > > in this case, you can always use wdog_periods_s with period == 0 for no period case So, to avoid similar problems, we must use wdog_period_s, right? could we remove wdog_s and only keep wdog_period_s? I don't think he needs to exist anymore -- 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-unsubscr...@nuttx.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org