Commit 81980c16 authored Dec 19, 2014 by Eduardo Barretto Committed by Greg Kroah-Hartman Jan 17, 2015

Staging: wlan-ng: hfa384x_usb: fixed an 'else' statement coding style issue



Removed useless 'else' statement that followed an 'if' statement that
had a return 1 and moved all the content from the 'else' to outside of
the switch case, this way if any case is sufficient it returns '1',
otherwise it will return 0.

Signed-off-by: Eduardo Barretto <edusbarretto@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

parent aa9895da

drivers/staging/wlan-ng/hfa384x_usb.c

+5 −6

Original line number	Diff line number	Diff line
		@@ -4123,12 +4123,11 @@ static int hfa384x_isgood_pdrcode(u16 pdrcode)
		pr_debug("Encountered unknown PDR#=0x%04x, assuming it's ok.\n",
		pdrcode);
		return 1;
		} else {
		}
		break;
		}
		/* bad code */
		pr_debug("Encountered unknown PDR#=0x%04x, (>=0x1000), assuming it's bad.\n",
		pdrcode);
		return 0;
		}
		}
		return 0; /* avoid compiler warnings */
		}