Paolo Bonzini wrote: > On 09/15/2011 12:27 PM, Jim Meyering wrote: >> From: Jim Meyering<[email protected]> >> >> * src/dfa.c (free_mbdata): Use regular "if", not #if MBS_SUPPORT. >> --- >> src/dfa.c | 47 ++++++++++++++++++++++++----------------------- >> 1 files changed, 24 insertions(+), 23 deletions(-) >> >> diff --git a/src/dfa.c b/src/dfa.c >> index 30013a2..acd99aa 100644 >> --- a/src/dfa.c >> +++ b/src/dfa.c >> @@ -3407,38 +3407,39 @@ dfaexec (struct dfa *d, char const *begin, char *end, >> } >> } >> >> -#if MBS_SUPPORT >> static void >> free_mbdata (struct dfa *d) >> { >> - unsigned int i; >> + if (MBS_SUPPORT) >> + { ... >> + } >> } >> -#endif >> >> /* Initialize the components of a dfa that the other routines don't >> initialize for themselves. */ > > Either here or later in the series, this if can be removed.
Good catch. I've removed it there -- and adjusted the log, of course.
