[PATCH v3] staging: rtl8723bs: remove redundant ternary operators from return
Ashmit Kumar <[email protected]>
| Newsgroups | gmane.linux.kernel |
|---|---|
| Message-ID | <[email protected]> |
The Linux kernel coding style specifies that boolean expressions do not need a redundant `? true : false` ternary operator. Additionally, return is not a function and therefore parentheses are not required. Signed-off-by: Ashmit Kumar <[email protected]> --- Changes in v3: - Formatted as a standalone patch (removed 1/3 series numbering). drivers/staging/rtl8723bs/os_dep/osdep_service.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/rtl8723bs/os_dep/osdep_service.c b/drivers/staging/rtl8723bs/os_dep/osdep_service.c index 4cfdf7c6234..214c1fd9b3e 100644 --- a/drivers/staging/rtl8723bs/os_dep/osdep_service.c +++ b/drivers/staging/rtl8723bs/os_dep/osdep_service.c @@ -130,7 +130,7 @@ void rtw_buf_update(u8 **buf, u32 *buf_len, u8 *src, u32 src_len) */ inline bool rtw_cbuf_full(struct rtw_cbuf *cbuf) { - return (cbuf->write == cbuf->read - 1) ? true : false; + return cbuf->write == cbuf->read - 1; } /** @@ -141,7 +141,7 @@ inline bool rtw_cbuf_full(struct rtw_cbuf *cbuf) */ inline bool rtw_cbuf_empty(struct rtw_cbuf *cbuf) { - return (cbuf->write == cbuf->read) ? true : false; + return cbuf->write == cbuf->read; } /** -- 2.51.0.windows.1