diff options
author | John Crispin <blogic@openwrt.org> | 2014-08-25 06:35:50 +0000 |
---|---|---|
committer | John Crispin <blogic@openwrt.org> | 2014-08-25 06:35:50 +0000 |
commit | e723b7928ac7f10adddcb6986930a4c0f8a7ac79 (patch) | |
tree | 9fdfe0ac58abb317b80df55e8715c220fd5b6d0c /package/network/services/relayd | |
parent | 6c505b7533f02bcf1baf9d00d2b885cb778b5d1a (diff) | |
download | upstream-e723b7928ac7f10adddcb6986930a4c0f8a7ac79.tar.gz upstream-e723b7928ac7f10adddcb6986930a4c0f8a7ac79.tar.bz2 upstream-e723b7928ac7f10adddcb6986930a4c0f8a7ac79.zip |
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 <bittorf@bluebottle.com>
git-svn-id: svn://svn.openwrt.org/openwrt/trunk@42278 3c298f89-4303-0410-b956-a3cf2f4a3e73
Diffstat (limited to 'package/network/services/relayd')
-rw-r--r-- | package/network/services/relayd/files/relay.init | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/package/network/services/relayd/files/relay.init b/package/network/services/relayd/files/relay.init index e193312e6e..e38143f6e7 100644 --- a/package/network/services/relayd/files/relay.init +++ b/package/network/services/relayd/files/relay.init @@ -15,8 +15,6 @@ validate_proto_relayd() 'table:range(0, 65535):16800' \ 'forward_bcast:bool:1' \ 'forward_dhcp:bool:1' - - return $? } resolve_ifname() { |