[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Please add comments to isearch.el
From: |
Artur Malabarba |
Subject: |
Re: Please add comments to isearch.el |
Date: |
Fri, 11 Dec 2015 16:00:27 +0000 |
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)
> 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.
- 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 <=
- 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, 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