Re: A simplification of _AC_CANONICAL_SPLIT

[Noah Misch]

On Tue, Aug 16, 2005 at 07:05:46PM +0200, Stepan Kasal wrote:
> --- lib/autoconf/general.m4   16 Aug 2005 09:11:06 -0000      1.880
> +++ lib/autoconf/general.m4   16 Aug 2005 16:49:15 -0000
> @@ -1659,19 +1659,14 @@
>  ac_save_IFS=$IFS; IFS='-'
>  set x $ac_cv_$1
>  shift
> -IFS=$ac_save_IFS
> -
>  AC_SUBST([$1_cpu], [$[1]])dnl
>  AC_SUBST([$1_vendor], [$[2]])dnl
> -AC_SUBST([$1_os], [$[3]])dnl
>  case $[#] in
> -0|1|2) ;;
> -*)
> -  shift; shift; shift
> -  for ac_arg
> -  do $1_os=[$]$1_os-$ac_arg
> -  done;;
> +  0|1) set "";;
> +  *)   shift; shift;;
>  esac

This case statement now merely avoids shifting an empty argument list.  Given
that this code splits canonical platform names, which have three or more fields,
why not just shift twice, unconditionally?

> +AC_SUBST([$1_os], [$[*]])dnl
> +IFS=$ac_save_IFS
>  ])# _AC_CANONICAL_SPLIT