Re: patch for UNINIT error in array.c

> Along with the unitialized beg, we found a len which
> doesn't appear to be initialized anywhere either.
> ...
> diff -p -u -r1.186 array.c
> --- array.c     12 Dec 2005 16:46:59 -0000      1.186
> +++ array.c     3 May 2006 02:22:50 -0000
> @@ -2102,6 +2102,8 @@ rb_ary_fill(int argc, VALUE *argv, VALUE
>      long beg, end, len;
>      VALUE *p, *pend;
>      int block_p = Qfalse;
> +    beg = 0;
> +    len = 0;
>
>      if (rb_block_given_p()) {
>         block_p = Qtrue;

    If you're going to initialize beg and len, you might as well
initialize end too.  Instead of adding initializer lines, I'd just
change:

    long beg, end, len;

    ...to:

    long beg = 0, end = 0, len = 0;

    ...to keep the style consistent.

    - Warren Brown