Re: [PR] sched: Fix typo on queue.h [nuttx]

2025-05-07 Thread via GitHub


acassis closed pull request #16340: sched: Fix typo on queue.h
URL: https://github.com/apache/nuttx/pull/16340


-- 
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]



Re: [PR] sched: Fix typo on queue.h [nuttx]

2025-05-07 Thread via GitHub


acassis commented on PR #16340:
URL: https://github.com/apache/nuttx/pull/16340#issuecomment-2861021208

   @hujun260 could you please review? As Serg noticed the issue wasn't evident 
before because "prev" was defined in other context and used from there. But we 
don't know it side effect this typo could cause in the kernel.


-- 
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]



[PR] sched: Fix typo on queue.h [nuttx]

2025-05-07 Thread via GitHub


acassis opened a new pull request, #16340:
URL: https://github.com/apache/nuttx/pull/16340

   ## Summary
   
   This typo was in the dq_insert_mid() macro introduced in the 
https://github.com/apache/nuttx/pull/13616
   
   This issue was found by Serg Podtynnyi 
   
   
   ## Impact
   
   Fix
   
   ## Testing
   
   Build
   


-- 
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]