Re: [Outreachy kernel] [PATCH 1/3] staging: rtl8723bs: core: fix repeated word warning
From: Julia Lawall <julia.lawall@inria.fr>
Date: 2021-03-31 20:18:29
Also in:
lkml
On Wed, 31 Mar 2021, Deborah Brouwer wrote:
Fix checkpatch warning: WARNING: Possible repeated word: 'very'
This is a simple but clear example of how "Fix" doesn't help one understand the patch. In reading the log message, one would probably assume that you removed the repetition, but actually you added a comma. So it would be better to explain what you did and why. It's good to acknowledge checkpatch, but the text of the warning message is not that useful. julia
quoted hunk ↗ jump to hunk
Signed-off-by: Deborah Brouwer <redacted> --- drivers/staging/rtl8723bs/core/rtw_xmit.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)diff --git a/drivers/staging/rtl8723bs/core/rtw_xmit.c b/drivers/staging/rtl8723bs/core/rtw_xmit.c index 2daf5c461a4d..3878caf0b56c 100644 --- a/drivers/staging/rtl8723bs/core/rtw_xmit.c +++ b/drivers/staging/rtl8723bs/core/rtw_xmit.c@@ -1700,7 +1700,7 @@ Calling context: If we turn on USE_RXTHREAD, then, no need for critical section. Otherwise, we must use _enter/_exit critical to protect free_xmit_queue... -Must be very very cautious... +Must be very, very cautious... */ struct xmit_frame *rtw_alloc_xmitframe(struct xmit_priv *pxmitpriv)/* _queue *pfree_xmit_queue) */ --2.17.1 -- You received this message because you are subscribed to the Google Groups "outreachy-kernel" group. To unsubscribe from this group and stop receiving emails from it, send an email to outreachy-kernel+unsubscribe@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/outreachy-kernel/14c1a1f1f0a34fb821d47ddab6e7e63800ec2736.1617221075.git.deborahbrouwer3563%40gmail.com.