bug#22338: 25.0.50; deactivate-mark regression

[Eli Zaretskii]

> From: Leo Liu <sdl.web@gmail.com>
> Date: Thu, 21 Jan 2016 18:13:36 +0800
> 
> How did you revert? C-x C-v or eval (revert-buffer nil t t)?

The latter, of course.

> For me the latter always keeps the region active in 25.0.50 (built
> on 02 Jan 2016).

My build is more current, but I don't think anything's changed since
Jan 2.

Would you mind showing a more detailed recipe, including the file you
used and the exact procedure to change the file behind Emacs's back?
Also, how do you check that the mark is still active after the revert?
I feel there are some fine details that are different between our
attempts to reproduce the issue, because I cannot reproduce it no
matter what I do.  There's some factor here that I don't get.  I hope
it's not some stupid oversight on my part.

> The trouble is now users have no idea what the active region is. For
> example I've experienced reverts that change the active region to cover
> nearly the whole buffer.

Well, if we are talking about users, why would a user invoke
revert-buffer in such a strange way?  And if the region is active, but
not highlighted, doesn't "C-x C-x" show it highlighted?

Anyway, let's delay this discussion until we understand what exactly
is needed to reproduce the issue.

Thanks.