On 23 December 2016 at 00:03, <npostavs@users.sourceforge.net> wrote:

> Reuben Thomas <rrt@sc3d.org> writes:
>
> > On Dec 22, 2016 2:17 AM, <npostavs@users.sourceforge.net> wrote:
> >> >
> >> > The only thing I'd change in your match is, as in mine, remove the
> FIXME after save-match-data.​
> >>
> >> (whitespace-cleanup): Remove a FIXME: the reason for save-match-data
> >> is the call of re-search-forward.
> >>
> >> I'm not sure this is a sufficient explanation of why save-match-data is
> >> needed. Usually in Emacs the caller is responsible for saving the match
> >> data, if it needs to. I don't really see the need here.
> >
> > Can it be removed then?
>
> Probably yes.


​Great; can you then install a suitable patch, please?

-- 
http://rrt.sc3d.org