Re: bug(s) in 2.5.19 skel.c

[Bruce Lilly]

Bruce Lilly wrote:
> A simpler and more effective alternative (since)
> only a few elements of the struct need to be
> initialized to something other thsn zero) would
> be either:
> 1. use calloc rather than malloc
> or
> 2. use memset() after malloc.
> Then yy_init_globals only has to take care of
> the elements that need to be initialized to non-xero
> values.

The attached patch implements the second option.



*** skel.c.orig Sat Sep 14 22:32:14 2002
--- skel.c      Sun Sep 15 14:05:24 2002
***************
*** 2020,2040 ****
    "#ifdef YY_USE_LINENO",
    "    yylineno =  1;",
    "#endif",
-   "    YY_G(yy_current_buffer) = 0;",
-   "    YY_G(yy_c_buf_p) = (char *) 0;",
    "    YY_G(yy_init) = 1;",
-   "    YY_G(yy_start) = 0;",
-   "    YY_G(yy_start_stack_ptr) = 0;",
-   "    YY_G(yy_start_stack_depth) = 0;",
-   "    YY_G(yy_start_stack) = (int *) 0;",
    "",
    "/* Defined in main.c */",
    "#ifdef YY_STDINIT",
    "    yyin = stdin;",
    "    yyout = stdout;",
-   "#else",
-   "    yyin = (FILE *) 0;",
-   "    yyout = (FILE *) 0;",
    "#endif",
    "",
    "    /* For future reference: Set errno on error, since we are called by",
--- 2020,2031 ----
***************
*** 2067,2072 ****
--- 2058,2064 ----
    "        return 1;",
    "    }",
    "",
+   "    memset(*ptr_yy_globals, 0, sizeof( struct yyguts_t ));",
    "    return yy_init_globals ( *ptr_yy_globals );",
    "    }",
    "",