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

need a reindex when reopen vim? #64

Closed
xltan opened this issue Sep 8, 2017 · 16 comments
Closed

need a reindex when reopen vim? #64

xltan opened this issue Sep 8, 2017 · 16 comments

Comments

@xltan
Copy link

xltan commented Sep 8, 2017

Greet Plugin! Thanks~
It seems the cache is not persistent when vim is closed, intended?

@Yggdroot
Copy link
Owner

Yggdroot commented Sep 8, 2017

Yes, if there is no rg or ag or pt or find, the cache will work.

@xltan
Copy link
Author

xltan commented Sep 8, 2017

Thanks for the prompt reply.
Another question about cache:
In some scenarios, one may work with multi directories, again, the reindex is needed there, but the cost may be intolerable.
Are there any plans to address this problem?

@Yggdroot
Copy link
Owner

Yggdroot commented Sep 8, 2017

OK, I'll add an option to specify whether to cache the filenames later.

@xltan
Copy link
Author

xltan commented Sep 8, 2017

Wow, you're so great, thanks again, btw, your code is really good and clean!

@xltan
Copy link
Author

xltan commented Sep 9, 2017

Hi again~
I find this snippet in your code, bufTagExpl\functionExpl, etc:
lfCmd("setlocal cursorline! | redraw | sleep 100m | setlocal cursorline!")
I think this cursorline trick can be a little annoying when in preview mode~

@Yggdroot
Copy link
Owner

Yggdroot commented Sep 9, 2017

let g:Lf_PreviewResult = { 'BufTag': 0, 'Function': 0 }, this can disable the preview.

@xltan
Copy link
Author

xltan commented Sep 9, 2017

yep~ at first glance, I thought the preview is really cool, but later, I find it will mess up with my jumplist, so I just disable them, thanks anyway.

Yggdroot added a commit that referenced this issue Sep 11, 2017
@Yggdroot
Copy link
Owner

Cache added. Please use the dev branch to have a try.

@xltan
Copy link
Author

xltan commented Sep 11, 2017

Work like a charm! Thanks

@xltan
Copy link
Author

xltan commented Sep 11, 2017

refresh the cache seems broken~

@xltan
Copy link
Author

xltan commented Sep 11, 2017

2 bugs:
when cache option turns on, after reopen vim, _buildCmd may not be processed, so no external cmd will be initialized.
getFreshContent should ignore cache.

@Yggdroot
Copy link
Owner

Yggdroot commented Sep 12, 2017

Thanks, I'll fix them soon.

Yggdroot added a commit that referenced this issue Sep 12, 2017
@Yggdroot
Copy link
Owner

fixed. Please have a try.

Yggdroot added a commit that referenced this issue Sep 12, 2017
@xltan
Copy link
Author

xltan commented Sep 12, 2017

cool, everything works very well.
btw, in the mean time, with little vim knowledge, I wrote a tag jump leaderf extension.
LeaderF-tjump: a simplified just work leaderf version vim-ctrlp-tjump.
just want to share with you that your extension is really easy to extend.

@Yggdroot
Copy link
Owner

Good, thanks.

Yggdroot added a commit that referenced this issue Sep 12, 2017
@sisrfeng
Copy link

sisrfeng commented Mar 2, 2022

Is there anything bad about this : LeaderF won't reindex the files when reopen vim

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

No branches or pull requests

3 participants