[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Please add comments to isearch.el
From: |
Eli Zaretskii |
Subject: |
Re: Please add comments to isearch.el |
Date: |
Fri, 11 Dec 2015 18:20:02 +0200 |
> Date: Fri, 11 Dec 2015 16:00:27 +0000
> From: Artur Malabarba <address@hidden>
> Cc: Alan Mackenzie <address@hidden>, emacs-devel <address@hidden>
>
> 2015-12-11 12:39 GMT+00:00 Eli Zaretskii <address@hidden>:
> > For example, start with isearch-mode, and then try to figure out what
> > each one of the following variables are in what use cases:
> > regexp-function and search-default-regexp-mode (as a function). You
> > will see that each one can be assigned to a variable that names a
> > function that has a default value that can be a function that...
>
> I didn't reply to your initial message for lack of time. I can add
> some comments here and there on the parts I understand. (some time in
> the coming months)
Thanks in advance.
> > Similarly with other isearch commands. This makes the source
> > impenetrable to uninitiated, much harder than just stepping with
> > Edebug through the code. IMO, if it is easier to understand code by
> > stepping through it in a debugger than by reading it, that code must
> > be refactored or documented the heck of.
>
> Honestly, I'd love to overhaul isearch. But I couldn't do it
> backwards-compatibly.
That's what master is for.
- Re: Please add comments to isearch.el, Eli Zaretskii, 2015/12/11
- Re: Please add comments to isearch.el, Alan Mackenzie, 2015/12/11
- Re: Please add comments to isearch.el, Eli Zaretskii, 2015/12/11
- Re: Please add comments to isearch.el, Artur Malabarba, 2015/12/11
- Re: Please add comments to isearch.el,
Eli Zaretskii <=
- Re: Please add comments to isearch.el, Artur Malabarba, 2015/12/11
- RE: Please add comments to isearch.el, Drew Adams, 2015/12/11
- Re: Please add comments to isearch.el, Eli Zaretskii, 2015/12/11
- Re: Please add comments to isearch.el, Juri Linkov, 2015/12/11
- Re: Please add comments to isearch.el, Artur Malabarba, 2015/12/11
- Re: Please add comments to isearch.el, Juri Linkov, 2015/12/11
- Re: Please add comments to isearch.el, Eli Zaretskii, 2015/12/12
- RE: Please add comments to isearch.el, Drew Adams, 2015/12/12
- Re: Please add comments to isearch.el, John Wiegley, 2015/12/12
- Re: Please add comments to isearch.el, Juri Linkov, 2015/12/12