summaryrefslogtreecommitdiff
path: root/package/system
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/system
parentfdc41e2fd79dac12f3c18aadef077f0f140f464c (diff)
downloadmtk-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/system')
-rw-r--r--package/system/ubox/files/log.init2
1 files changed, 0 insertions, 2 deletions
diff --git a/package/system/ubox/files/log.init b/package/system/ubox/files/log.init
index be38297..3e06fa5 100644
--- a/package/system/ubox/files/log.init
+++ b/package/system/ubox/files/log.init
@@ -19,7 +19,6 @@ validate_log_section()
'log_port:port:514' \
'log_proto:or("tcp", "udp"):udp' \
'log_prefix:string'
- return $?
}
validate_log_daemon()
@@ -27,7 +26,6 @@ validate_log_daemon()
uci_validate_section system system "${1}" \
'log_size:uinteger:0' \
'log_buffer_size:uinteger:0'
- return $?
}
start_service_daemon()