On Friday, June 03, 2011 14:36:03 Matthew McClintock wrote:
> +if [ -z "${options}" ] ; then
> +     options=$EXTRA_OPTS
> +else
> +     options="$options,$EXTRA_OPTS"
> +fi

this can be simplified (or at least done in one line) with:
options=${options:+${options},}${EXTRA_OPTS}

although i can understand that not everyone can grok that at first blush

> +TARGETS+=`echo $EXTRA_OPTS | tr ',' ' '`

you're still using non-portable += append operator

although i dont understand how EXTRA_OPTS can both be extra options and extra 
targets at the same time ...
-mike

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to