GUIDINGLI commented on PR #17642: URL: https://github.com/apache/nuttx/pull/17642#issuecomment-3717552276
> > I said yes to this PR, but still said no to : > > #17517 > > Although it has merged. > > It was not until this PR was merged that the quality of the HRTimer met the required standards. > > @anchao @wangchdo > > The direct merge of part1 PR was made in an unacceptably hasty manner. > > @GUIDINGLI @Fix-Point > > The PR received many-approving from PMCs and committers, I don't think there is any problem to merge it firstly. We can working to improve it. It was also improvement for my initial PR three months ago > > Besides, I am too busy recently to work on this any more, Looking forward to your continue work on queue abstraction for hrtimer. Why was the code merged directly when someone had pointed out bugs in it? It should only be merged after the bugs are fixed – especially given that there were still "Request changes" reviews pending. -- 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]
