On 02/15/2012 10:54 AM, Aharon Robbins wrote: > - int c, c1, c2; > + int c = 0, c1 = 0, c2 = 0;
Sorry, I don't see why this patch (and the other) is needed. grep's copy of src/dfa.h says this: extern void dfaerror (const char *) __attribute__ ((noreturn)); and all paths to the first use of c or wc lead through either an initialization of them, or a call to dfaerror. Also, all def-use paths for c1, c2, wc1, and wc2 seem sound, at least in grep's copy of dfa.c, so why do they need to be initialized?
