Commit dd850351 authored Oct 26, 2019 by Michael Straube Committed by Greg Kroah-Hartman Oct 26, 2019

staging: rtl8188eu: remove ternary operator



Instead of using ternary operator to set variable res, use the value
of variable match (or the negation) directly to simplify the code and
improve readability.

Signed-off-by: Michael Straube <straube.linux@gmail.com>
Link: https://lore.kernel.org/r/20191026121135.181897-5-straube.linux@gmail.com


Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

parent 4e6f391f

drivers/staging/rtl8188eu/core/rtw_sta_mgt.c

+2 −2

Original line number	Diff line number	Diff line
		@@ -504,9 +504,9 @@ bool rtw_access_ctrl(struct adapter padapter, u8 mac_addr)
		spin_unlock_bh(&pacl_node_q->lock);

		if (pacl_list->mode == 1)/* accept unless in deny list */
		res = (match) ? false : true;
		res = !match;
		else if (pacl_list->mode == 2)/* deny unless in accept list */
		res = (match) ? true : false;
		res = match;
		else
		res = true;

Admin message