diff options
| author | John Crispin <blogic@openwrt.org> | 2014-09-01 13:21:54 +0000 |
|---|---|---|
| committer | John Crispin <blogic@openwrt.org> | 2014-09-01 13:21:54 +0000 |
| commit | 78afb6f6f698dfa245da8a4de1fec51234e0e67c (patch) | |
| tree | ed4d2fed027aa3e5eac109458d8e29552d5b3826 /package/system/ubox/files/log.init | |
| parent | 0669f69d5c953b659b89e07e7ce333f8b85fec11 (diff) | |
| download | upstream-78afb6f6f698dfa245da8a4de1fec51234e0e67c.tar.gz upstream-78afb6f6f698dfa245da8a4de1fec51234e0e67c.tar.bz2 upstream-78afb6f6f698dfa245da8a4de1fec51234e0e67c.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>
Backport of r42278
git-svn-id: svn://svn.openwrt.org/openwrt/branches/barrier_breaker@42379 3c298f89-4303-0410-b956-a3cf2f4a3e73
Diffstat (limited to 'package/system/ubox/files/log.init')
| -rw-r--r-- | package/system/ubox/files/log.init | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/package/system/ubox/files/log.init b/package/system/ubox/files/log.init index be382972e3..3e06fa5b49 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() |