Author: dougb
Date: Sat Jan 14 08:59:02 2012
New Revision: 230103
URL: http://svn.freebsd.org/changeset/base/230103

Log:
  Now that its callers have been udpated, remove set_rcvar().
  
  The concept of set_rcvar() was nice in theory, but the forks
  it creates are a drag on the startup process, which is especially
  noticeable on slower systems, such as embedded ones.

Modified:
  head/etc/rc.subr

Modified: head/etc/rc.subr
==============================================================================
--- head/etc/rc.subr    Sat Jan 14 08:11:40 2012        (r230102)
+++ head/etc/rc.subr    Sat Jan 14 08:59:02 2012        (r230103)
@@ -54,47 +54,6 @@ JID=`$PS -p $$ -o jid=`
 #      functions
 #      ---------
 
-# set_rcvar [var] [defval] [desc]
-#
-#      Echo or define a rc.conf(5) variable name.  Global variable
-#      $rcvars is used.
-#
-#      If no argument is specified, echo "${name}_enable".
-#
-#      If only a var is specified, echo "${var}_enable".
-#
-#      If var and defval are specified, the ${var} is defined as
-#      rc.conf(5) variable and the default value is ${defvar}.  An
-#      optional argument $desc can also be specified to add a
-#      description for that.
-#
-set_rcvar()
-{
-       case $# in
-       0)
-               echo ${name}_enable
-               ;;
-       1)
-               echo ${1}_enable
-               ;;
-       *)
-               debug "rcvar_define: \$$1=$2 is added" \
-                   " as a rc.conf(5) variable."
-
-               local _var
-               _var=$1
-               rcvars="${rcvars# } $_var"
-               eval ${_var}_defval=\"$2\"
-               shift 2
-               # encode multiple lines of _desc
-               for l in "$@"; do
-                       eval ${_var}_desc=\"\${${_var}_desc#^^}^^$l\"
-               done
-               eval ${_var}_desc=\"\${${_var}_desc#^^}\"
-               ;;
-       esac
-}
-
 # set_rcvar_obsolete oldvar [newvar] [msg]
 #      Define obsolete variable.
 #      Global variable $rcvars_obsolete is used.
@@ -856,8 +815,8 @@ $command $rc_flags $command_args"
                                echo ""
                        fi
                        echo "#"
-                       # Get unique vars in $rcvar $rcvars
-                       for _v in $rcvar $rcvars; do
+                       # Get unique vars in $rcvar
+                       for _v in $rcvar; do
                                case $v in
                                $_v\ *|\ *$_v|*\ $_v\ *) ;;
                                *)      v="${v# } $_v" ;;
@@ -1003,7 +962,7 @@ run_rc_script()
 
        unset   name command command_args command_interpreter \
                extra_commands pidfile procname \
-               rcvar rcvars rcvars_obsolete required_dirs required_files \
+               rcvar rcvars_obsolete required_dirs required_files \
                required_vars
        eval unset ${_arg}_cmd ${_arg}_precmd ${_arg}_postcmd
 
@@ -1035,7 +994,7 @@ run_rc_script()
 #
 load_rc_config()
 {
-       local _name _var _defval _v _msg _new
+       local _name _rcvar_val _var _defval _v _msg _new
        _name=$1
        if [ -z "$_name" ]; then
                err 3 'USAGE: load_rc_config name'
@@ -1060,7 +1019,7 @@ load_rc_config()
        fi
 
        # Set defaults if defined.
-       for _var in $rcvar $rcvars; do
+       for _var in $rcvar; do
                eval _defval=\$${_var}_defval
                if [ -n "$_defval" ]; then
                        eval : \${$_var:=\$${_var}_defval}
_______________________________________________
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"

Reply via email to