Fix nesting/indent level whitespace mismatch.

The EndData() doesn't make sense as part of the condition that implies
the state is such that it's not inside data.  It might make sense as
part of an else block, but it also seems fine to unconditionally
EndData() like it currently does.  That way unexpected states (mail !=
0) are dealt with sooner rather than later.