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.

Sign up for GitHub

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

Jump to bottom

Add debug manual to README #81

Closed
stardiviner opened this issue Jan 20, 2016 · 3 comments
Closed

Add debug manual to README #81

stardiviner opened this issue Jan 20, 2016 · 3 comments

Comments

@stardiviner
Copy link

I found robe is very slow (about 4 seconds) when completing. I hope you can add a manual in README about how to debug robe deeply.
@dgutov
Copy link
Owner

dgutov commented Jan 20, 2016

debug manual

You read the README, read the code, understand it all thoroughly, and debug it. ;-)

Completion taking 4 seconds is possible if the prefix is empty, and Robe doesn't know the type of the target (so it returns all methods from all classes). That's why #26 is still open.

@dgutov dgutov closed this as completed Jan 20, 2016
dgutov added a commit that referenced this issue Jan 21, 2016
@dgutov
Visor: Skip singleton classes here too
0640fde 
The difference in performance is quite noticeable.

Not to mention that the scanners cover singleton classes already.

#81 is possibly related.
@dgutov
Copy link
Owner

dgutov commented Jan 21, 2016

@stardiviner Check out the latest commit. It should improve on those 4 seconds.

@stardiviner
Copy link
Author

I tried out the latest commit, it does speed up. Thanks very much.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@dgutov @stardiviner