From 2ae05c57f8d4cb557a5ea733c2795b8462f00062 Mon Sep 17 00:00:00 2001 From: John Crispin Date: Mon, 25 Aug 2014 06:35:50 +0000 Subject: package/*: remove useless explicit set of function returncode somebody started to set a function returncode in the validation stuff and everybody copies it, e.g. myfunction() { fire_command return $? } a function automatically returns with the last returncode, so we can safely remove the command 'return $?'. reference: http://tldp.org/LDP/abs/html/exit-status.html "The last command executed in the function or script determines the exit status." Signed-off-by: Bastian Bittorf SVN-Revision: 42278 --- package/network/config/netifd/files/etc/init.d/network | 14 -------------- 1 file changed, 14 deletions(-) (limited to 'package/network/config/netifd') diff --git a/package/network/config/netifd/files/etc/init.d/network b/package/network/config/netifd/files/etc/init.d/network index 8a7427bacd..1d8a5b34ef 100755 --- a/package/network/config/netifd/files/etc/init.d/network +++ b/package/network/config/netifd/files/etc/init.d/network @@ -50,8 +50,6 @@ validate_atm_bridge_section() 'atmdev:uinteger:0' \ 'encaps:or("llc", "vc"):llc' \ 'payload:or("bridged", "routed"):bridged' - - return $? } validate_route_section() @@ -64,8 +62,6 @@ validate_route_section() 'metric:uinteger' \ 'mtu:uinteger' \ 'table:or(range(0,65535),string)' - - return $? } validate_route6_section() @@ -77,8 +73,6 @@ validate_route6_section() 'metric:uinteger' \ 'mtu:uinteger' \ 'table:or(range(0,65535),string)' - - return $? } validate_rule_section() @@ -94,8 +88,6 @@ validate_rule_section() 'lookup:or(range(0,65535),string)' \ 'goto:range(0,65535)' \ 'action:or("prohibit", "unreachable", "blackhole", "throw")' - - return $? } validate_rule6_section() @@ -111,8 +103,6 @@ validate_rule6_section() 'lookup:or(range(0,65535),string)' \ 'goto:range(0,65535)' \ 'action:or("prohibit", "unreachable", "blackhole", "throw")' - - return $? } validate_switch_section() @@ -122,8 +112,6 @@ validate_switch_section() 'enable:bool' \ 'enable_vlan:bool' \ 'reset:bool' - - return $? } validate_switch_vlan() @@ -132,8 +120,6 @@ validate_switch_vlan() 'device:string' \ 'vlan:uinteger' \ 'ports:list(ports)' - - return $? } service_triggers() -- cgit v1.2.3