[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [elpa] master 5640cd0 04/12: el-search: some comment changes
From: |
Artur Malabarba |
Subject: |
Re: [elpa] master 5640cd0 04/12: el-search: some comment changes |
Date: |
Mon, 12 Oct 2015 00:29:57 +0100 |
On 11 Oct 2015 11:33 am, "Michael Heerdegen" <address@hidden> wrote:
> + ;; FIXME: an interesting alternative would be to really integrate it
> + ;; with Isearch, using `isearch-search-fun-function'.
> + ;; Alas, this is not trivial if we want to transfer our optimizations.
> (interactive)
> (el-search-pattern
> (el-search--read-pattern
Can you explain why? I've been thinking of making some refactorings to isearch (after the feature freeze) and I could try to extend isearch-search-fun-function to cover your use case if it's not too hard.
Best,
Artur
- [elpa] master updated (c0489e7 -> 2d5b3ba), Michael Heerdegen, 2015/10/11
- [elpa] master 5640cd0 04/12: el-search: some comment changes, Michael Heerdegen, 2015/10/11
- Re: [elpa] master 5640cd0 04/12: el-search: some comment changes,
Artur Malabarba <=
- [elpa] master 84282f0 08/12: avoid repeated expansion of pcase forms, Michael Heerdegen, 2015/10/11
- [elpa] master e1903f4 01/12: avoid looking-back in el-search--goto-next-sexp, Michael Heerdegen, 2015/10/11
- [elpa] master 855461b 03/12: rename a function, Michael Heerdegen, 2015/10/11
- [elpa] master ca3fe58 07/12: rename a function, Michael Heerdegen, 2015/10/11
- [elpa] master fa900da 09/12: new function el-search--end-of-sexp; use it, Michael Heerdegen, 2015/10/11
- [elpa] master ae052ce 11/12: rename a local variable, Michael Heerdegen, 2015/10/11
- [elpa] master 06c2076 02/12: change default of el-search-this-expression-identifier, Michael Heerdegen, 2015/10/11
- [elpa] master 5dd08f2 05/12: fix a condition in el-search--do-subsexps, Michael Heerdegen, 2015/10/15
- [elpa] master 0f79c16 06/12: drop el-search-expression-contains-match-p, Michael Heerdegen, 2015/10/15
- [elpa] master 2d5b3ba 12/12: whitespace clean up, Michael Heerdegen, 2015/10/15