Hi!
> +CASETYPE=$1
> +CASECMD=""
> +cnt=1
> +for arg; do
> +     if [ $cnt -gt 1 ]; then
> +             CASECMD+=" "
> +             CASECMD+=$arg
>       fi
> +     cnt=$(( $cnt + 1 ))
>  done

There is no need to do this. Just use the $@ instead of $CASECMD below.

> +case $CASETYPE in
> +"function" )
> +     : $(( CASENO1 += 1 ))
> +     export CASENO1=$CASENO1
> +     $TESTROOT/cgroup_fj_function.sh $CASECMD
> +     ;;
> +"function2" )
> +     : $(( CASENO1 += 1 ))
> +     export CASENO1=$CASENO1
> +     $TESTROOT/cgroup_fj_function2.sh $CASECMD
> +     ;;
> +"stress" )
> +     : $(( CASENO2 += 1 ))
> +     export CASENO2=$CASENO2
> +     $TESTROOT/cgroup_fj_stress.sh $CASECMD
> +     ;;
> +esac
> +
> +ret=$?
> +if [ $ret -eq 0 ]; then
> +     tst_resm TPASS "case$i($CASETYPE$CASECMD)    PASS"
> +elif [ $ret -ne 9 ]; then
> +     tst_resm TFAIL "case$i($CASETYPE$CASECMD)    FAIL"
> +fi

The rest looks good to me.

-- 
Cyril Hrubis
chru...@suse.cz

------------------------------------------------------------------------------
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

Reply via email to