Subject: Re: bug#43714: 28.1: auto-revert code improvements [PATCH]

From: Eli Zaretskii <[email-masked]>
Date: Wed, 30 Sep 2020 19:03:55 +0300

> Date: Wed, 30 Sep 2020 11:28:27 -0400
> From: Boruch Baum <[email-masked]>
> Cc: Lars Ingebrigtsen <[email-masked]>, [email-masked]
> 
> On 2020-09-30 17:25, Eli Zaretskii wrote:
> > >     (auto-revert-buffers): Use save-match-data.
> >
> > It could be due to remote files (and file-handlers)?
> 
> But I don't see anywhere in the code itself being evaluated in
> auto-revert.el that would produce its own match-data to potentially
> disturb the pre-existing state.

That's what I meant: it could be hidden from plain sight.

Why is that a problem to leave save-match-data alone?  Does it do any
harm?