summaryrefslogtreecommitdiffstats
path: root/package/utils/busybox
diff options
context:
space:
mode:
authorJohn Crispin <john@openwrt.org>2014-08-25 06:35:50 +0000
committerJohn Crispin <john@openwrt.org>2014-08-25 06:35:50 +0000
commit2ae05c57f8d4cb557a5ea733c2795b8462f00062 (patch)
tree8f54c6b5b6868ff35369f650de1a01f7eef4a296 /package/utils/busybox
parentfdc41e2fd79dac12f3c18aadef077f0f140f464c (diff)
downloadmaster-31e0f0ae-2ae05c57f8d4cb557a5ea733c2795b8462f00062.tar.gz
master-31e0f0ae-2ae05c57f8d4cb557a5ea733c2795b8462f00062.tar.bz2
master-31e0f0ae-2ae05c57f8d4cb557a5ea733c2795b8462f00062.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> SVN-Revision: 42278
Diffstat (limited to 'package/utils/busybox')
-rwxr-xr-xpackage/utils/busybox/files/sysntpd1
1 files changed, 0 insertions, 1 deletions
diff --git a/package/utils/busybox/files/sysntpd b/package/utils/busybox/files/sysntpd
index 1878c3f594..417addfb1c 100755
--- a/package/utils/busybox/files/sysntpd
+++ b/package/utils/busybox/files/sysntpd
@@ -9,7 +9,6 @@ PROG=/usr/sbin/ntpd
validate_ntp_section() {
uci_validate_section system timeserver "${1}" \
'server:list(host)' 'enabled:bool:1' 'enable_server:bool:0'
- return $?
}
start_service() {