Changeset 36298


Ignore:
Timestamp:
2013-04-10T12:48:32+02:00 (5 years ago)
Author:
zioproto
Message:

[packages] olsrd: init: for better reuse, make a function out of a procedure get_value_for_entry()

Later we want to reuse that functionality, to avoid code duplication
we make a function with clean entry and return-codes, so we can do e.g.:

get_value_for_entry "$entry"
continue

which is also better readable.

Signed-off-by: Bastian Bittorf <bittorf@…>

File:
1 edited

Legend:

Unmodified
Added
Removed
  • packages/net/olsrd/files/olsrd.init

    r35680 r36298  
    233233        [ -z "$write_func" ] && output_func=echo 
    234234        local write_param="$4" 
     235 
    235236        local schema_entry 
    236237        local option 
     
    245246        local position 
    246247 
    247         for schema_entry in $schema; do 
     248        get_value_for_entry() 
     249        { 
     250                local schema_entry="$1" 
     251 
    248252                default="${schema_entry#*[=]}" 
    249253                [ "$default" = "$schema_entry" ] && default= 
    250254                option="${schema_entry%%[=]*}" 
    251                 IFS=':' 
    252                 set -- $option 
    253                 unset IFS 
     255 
     256                IFS=':'; set -- $option; unset IFS 
    254257                option="$1" 
    255258                option_type="$2" 
    256                 validate_varname "$option" || continue 
    257                 [ -z "$option_type" ] || validate_varname "$option_type" || continue 
    258                 [ "$option_type" = internal ] && continue 
     259 
     260                validate_varname "$option" || return 1 
     261                [ -z "$option_type" ] || validate_varname "$option_type" || return 1 
     262                [ "$option_type" = internal ] && return 1 
     263 
    259264                config_get value "$cfg" "$option" 
     265 
     266                return 0 
     267        } 
     268 
     269        for schema_entry in $schema; do 
     270                get_value_for_entry "$schema_entry" || continue 
    260271 
    261272                if [ -z "$value" ]; then 
     
    482493        ifnames= 
    483494        config_get interfaces "$cfg" interface 
     495 
    484496        for interface in $interfaces; do 
    485497                if validate_varname "$interface"; then 
Note: See TracChangeset for help on using the changeset viewer.