diff options
author | John Crispin <john@openwrt.org> | 2014-08-25 06:35:50 +0000 |
---|---|---|
committer | John Crispin <john@openwrt.org> | 2014-08-25 06:35:50 +0000 |
commit | 2ae05c57f8d4cb557a5ea733c2795b8462f00062 (patch) | |
tree | 8f54c6b5b6868ff35369f650de1a01f7eef4a296 /package/network/services/dropbear/files/dropbear.init | |
parent | fdc41e2fd79dac12f3c18aadef077f0f140f464c (diff) | |
download | mtk-20170518-2ae05c57f8d4cb557a5ea733c2795b8462f00062.zip mtk-20170518-2ae05c57f8d4cb557a5ea733c2795b8462f00062.tar.gz mtk-20170518-2ae05c57f8d4cb557a5ea733c2795b8462f00062.tar.bz2 |
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>
SVN-Revision: 42278
Diffstat (limited to 'package/network/services/dropbear/files/dropbear.init')
-rwxr-xr-x | package/network/services/dropbear/files/dropbear.init | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/package/network/services/dropbear/files/dropbear.init b/package/network/services/dropbear/files/dropbear.init index 5206426..c6f3d9b 100755 --- a/package/network/services/dropbear/files/dropbear.init +++ b/package/network/services/dropbear/files/dropbear.init @@ -47,7 +47,6 @@ validate_section_dropbear() 'Port:list(port):22' \ 'SSHKeepAlive:uinteger:300' \ 'IdleTimeout:uinteger:0' - return $? } dropbear_instance() |