Re: replace.el changes...

[David Kastrup]

Juri Linkov <address@hidden> writes:

> David Kastrup <address@hidden> writes:
> > If a replacement has already been done, the stored information in the
> > history does not contain the highlighted region.  So I am going to
> > have to either change the history information format, or don't enter
> > actually performed replacements into that list for revisiting in the
> > first place, or don't highlight things when going backwards in the
> > list.  Incidentally, the history is also the return value from all
> > replacement commands: anybody know whether this is actually used
> > anywhere?
> 
> I don't know where it is used, but I think it's better not to change
> the format of the return value.

It is actually not documented to be useful, except as being nil or
non-nil.  And even then, it is only documented for perform-replace
itself (the internal routine doing the job).  AFAICS, the return value
from perform-replace is actually used nowhere within Emacs directly
_except_ for incidentally also being the return value of all
replacement-related functions.  Searching for uses of all of them
would be work...

> You could remove your additional fields from the history before
> returning it.

Sounds like overkill for something that is not even documented to be
useful.

-- 
David Kastrup, Kriemhildstr. 15, 44793 Bochum