Vijay Banerjee commented: 
https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1363#note_155219


Hi @Chloe. Thanks for the MR.

We generally treat the commit message to talk about what the commit message is 
doing, instead of what we did in the commit.

So, instead of "Therefore, I update the code to explicitly check for NULL 
before using the descriptor.", you can just write: "Add check for NULL pointer 
after malloc call", or something similar to that

-- 
View it on GitLab: 
https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1363#note_155219
You're receiving this email because of your account on gitlab.rtems.org. 
Unsubscribe from this thread: 
https://gitlab.rtems.org/-/sent_notifications/4-cjhdw4i63ni7e9hichedjnmjb-1d/unsubscribe
 | Manage all notifications: https://gitlab.rtems.org/-/profile/notifications | 
Help: https://gitlab.rtems.org/help


_______________________________________________
bugs mailing list
[email protected]
http://lists.rtems.org/mailman/listinfo/bugs

Reply via email to