Re: [PATCH] Fix compat/regex ANSIfication on MinGW

Ævar Arnfjörð Bjarmason [off-list ref] writes:

Yeah, wasn't the only reason they were ANSI-fied because Junio had
some script in the todo branch that explicitly used -Werror in
conjunction with a warning for k&r declerations/

As a matter of principle, I'd really want my compilation to pass without
any warning, so -Werror is likely to stay.

Having said that, the codepath in question is not something I'd be
compiling myself very often, so I am not in a good position to vetting and
maintaining the particular ANSIification patch.  Personally I think the
change to move "internal_function" to the front (which Karsten mentioned
is consistent with the way how the same issue is handled elsewhere in
msysgit port) would be cleaner than the change to remove it.

I'd ask folks involved in Windows port to decide.  Sorry for the trouble.