Messages in this thread | | | From | Austin Kim <> | Date | Thu, 28 Nov 2019 07:21:40 +0900 | Subject | Re: [PATCH] brcmsmac: Remove always false 'channel < 0' statement |
| |
2019년 11월 27일 (수) 오후 10:35, Kalle Valo <kvalo@codeaurora.org>님이 작성: > > Austin Kim <austindh.kim@gmail.com> writes: > > > 2019년 11월 27일 (수) 오후 7:48, Sergei Shtylyov > > <sergei.shtylyov@cogentembedded.com>님이 작성: > >> > >> On 27.11.2019 8:43, Austin Kim wrote: > >> > >> > As 'channel' is declared as u16, the following statement is always false. > >> > channel < 0 > >> > > >> > So we can remove unnecessary 'always false' statement. > >> > >> It's an expression, not a statement. > >> > > > > According to below link, it is okay to use 'statement' in above case. > > https://en.wikipedia.org/wiki/Statement_(computer_science) > > I don't have time to start arguing about this, and I'm no C language > lawyer either, but all I say is that I agree with Sergei here.
Thanks for your opinion. I will use 'expression' rather than 'statement' when I upstream similar patch later.
> > > Why don't you show your opition about patch rather than commit message? > > But this comment is not ok. Patch review (including commit logs) is the > core principle of upstream development so you need to have an open mind > for all comments, even the ones you don't like.
Oh! I Agreed. If I were you, I would leave similar comment.
Thanks, Austin Kim
> > -- > https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
| |