New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Docstring based search for ac-isearch #234

Open
terranpro opened this Issue May 6, 2013 · 2 comments

Comments

Projects
None yet
3 participants
@terranpro

terranpro commented May 6, 2013

I found myself wanting to filter completion results by docstring, and thought this might be useful functionality for others too. It looks like it needs to be provided by popup-el, so I have suggested a patch for popup-el, and a sample defun for invocation w/auto-complete.

Please see:

auto-complete/popup-el#39
auto-complete/popup-el/pull/40

@m2ym

This comment has been minimized.

Show comment
Hide comment
@m2ym

m2ym May 11, 2013

Member

Please let me think the patch should be merged or not into version 1.x. I'd ratherlike to implement such a feature in v2.0...

Member

m2ym commented May 11, 2013

Please let me think the patch should be merged or not into version 1.x. I'd ratherlike to implement such a feature in v2.0...

@tkf

This comment has been minimized.

Show comment
Hide comment
@tkf

tkf Jun 11, 2013

Member

I am not sure if it should be in popup.el. For such complex search I think it is a job for helm. See: https://github.com/yasuyk/ac-helm

When the list is large, and if you want to search docstring, you would need some kind of async search to make isearch responsive. Async search is already implemented in helm. So why not use it?

Member

tkf commented Jun 11, 2013

I am not sure if it should be in popup.el. For such complex search I think it is a job for helm. See: https://github.com/yasuyk/ac-helm

When the list is large, and if you want to search docstring, you would need some kind of async search to make isearch responsive. Async search is already implemented in helm. So why not use it?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment