contributed by < jouae >
Reviewed by SimonLiu423
Commit fc06cb9 中同時修改了 q_new, q_free, q_insert_*, q_size 函式,然而這幾個函式並沒有都彼此相關,全部放在同個 commit 容易造成閱讀的人混淆,建議將 commit 細拆成更多個,例如:q_new, q_free: 與 queue 配置、釋放相關的放一個 commit
q_insert_*: 與新增元素至 queue 相關的放一個 commit
q_size: 獨自一個 commit
此外,commit message 中有提到 malloc 的 reference, 建議可以直接將連結放在 commit message 的末端: Link: <...>,確保讀者跟你看的 refenrece 是同一個。
對於 queue.[ch] 實作的說明,建議可以貼出部分程式碼好讓讀者知道你在指哪段程式碼,例如在 q_remove_head, q_remove_tail: