wangchdo commented on PR #17642: URL: https://github.com/apache/nuttx/pull/17642#issuecomment-3718090777
> > > 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. > > @wangchdo 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. The merged PR is not the one that has"Request changes" reviews pending. Anyway this is not important, the important thing is that I didn't receive any comments from you guys in the PR merged, And I only received your guys' showing off saying that your implementation was better in every aspect than mine in a general way. Besides, I did provide over 20 comments in your guys' implementation afterwords pointing out the issues specifically. Without any showing off in a general way. Also I added coauthor of Fix-Point in my second PR even though the design is completely different than what you guys want and you guys have been criticizing it and pushing me to revert it with yours constantly. I think this is end. -- 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]
