Skip to content
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

faster ctrlp-cmatcher #326

Closed
skwp opened this issue May 7, 2013 · 7 comments
Closed

faster ctrlp-cmatcher #326

skwp opened this issue May 7, 2013 · 7 comments

Comments

@skwp
Copy link
Owner

skwp commented May 7, 2013

I just locally installed https://github.com/JazzCore/ctrlp-cmatcher

It works much faster than the normal ctrlp matcher, because it uses a C extension. I'm going to use it locally for a while before trying to integrate.

The biggest issue here is it requires local compilation, so having python installed. I'm not sure it's a great thing to add such a dependency to yadr or how to integrate it into the Rakefile. Additionally you have to manually compile it (see their README) and then copy the .so file: JazzCore/ctrlp-cmatcher#3

@lfilho
Copy link
Collaborator

lfilho commented May 7, 2013

Personally, the current speed of CtrlP is enough for me. I have a considerably big project here and the speed doesn't interfere with my current workflow...

And I remember I was pretty annoyed when I used Command-T and also had to compile it everytime I reinstalled or cloned my repo in a new machine, etc etc etc...

@duhanebel
Copy link
Collaborator

Installation is not a big deal: we can kick off the compilation from the rake file and install it where it needs to go after we run BundleInstall.

Fabio

On 7 May 2013, at 16:09, Yan Pritzker [email protected] wrote:

I just locally installed https://github.com/JazzCore/ctrlp-cmatcher

It works much faster than the normal ctrlp matcher, because it uses a C extension. I'm going to use it locally for a while before trying to integrate.

The biggest issue here is it requires local compilation, so having python installed. I'm not sure it's a great thing to add such a dependency to yadr or how to integrate it into the Rakefile. Additionally you have to manually compile it (see their README) and then copy the .so file: JazzCore/ctrlp-cmatcher#3


Reply to this email directly or view it on GitHub.

@lfilho
Copy link
Collaborator

lfilho commented May 7, 2013

If we turned out using it (the fully automated installation and compilation Fabio mentioned made me smile), then the python dependecy could open the doors for using UltiSnips in place of SnipMate.

There's a huge discussion here where the author of SnipMate himself proposes the "end" of the project in favor of UltiSnips: garbas/vim-snipmate#114
And maybe this is useful too: http://fueledbylemons.com/blog/2011/07/27/why-ultisnips/
(And honza/vim-snippets are already compatible with both)

@duhanebel
Copy link
Collaborator

I hope that the smiling thing was a good thing :P
The new powerline toolbar is also gonna be written in python so I suppose we'll need the dependency anyway in the future.

Fabio

On 7 May 2013, at 16:48, lfilho [email protected] wrote:

If we turned out using it (the fully automated installation and compilation Fabio mentioned made me smile), then the python dependecy could open the doors for using UltiSnips in place of SnipMate.

There's a huge discussion here where the author of SnipMate himself proposes the "end" of the project in favor of UltiSnips: garbas/vim-snipmate#114
And maybe this is useful too: http://fueledbylemons.com/blog/2011/07/27/why-ultisnips/
(And honza/vim-snippets are already compatible with both)


Reply to this email directly or view it on GitHub.

@lfilho
Copy link
Collaborator

lfilho commented May 7, 2013

Hehehe yeah, that was not well expressed, eh?
I meant that it made me happy. Because I just wouldn't like to be compiling stuff every time...

@duhanebel
Copy link
Collaborator

@lfilho: If there's a thing my job taught me is that there's always a way to automate stuff ;)

@skwp skwp closed this as completed Aug 4, 2013
@skwp
Copy link
Owner Author

skwp commented Aug 4, 2013

I'm only closing this because there's no action for 3 months. Ctrlp doesn't bother me, if it bothers you guys enough to try to do a pull request, I'm open to trying it out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants