Should we "fix" this declaration?

[Bart Schaefer <schaefer@xxxxxxxxxxxxxxxx>]

/**/
void
forget_edits(void)
{
    if (!have_edits) {
       return;
    }
    have_edits = 0;
    Histent he;

That's from users/29175, but we've previously followed the old C
format that declarations have to be at the start of a block.  If so,
it should be

/**/
void
forget_edits(void)
{
    Histent he;
    if (!have_edits) {
       return;
    }
    have_edits = 0;