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

Fix s:ExprCol() sometimes go into infinity loop when calculating indent #1137

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

shirohana
Copy link

I have no idea how to reproduce it in a simple way...

In the case, I'm working with several tabs and buffers,
when I edit a .vue file, I typing o (or CR, O, S, cc that will insert a newline), it happened.

But when I open only one .vue file, that will fall into infinity loop before, it works correctly...

And not only .vue but .js could cause the same error when lots of buffer was opened.

Sorry for no helpful information about it (ノω・。)

2018-07-28 7 26 48

2018-07-28 7 27 22

@shirohana
Copy link
Author

I think I didn't fix it correctly, hope you help (ノω・。)

@bounceme bounceme self-assigned this Oct 26, 2018
@bounceme
Copy link
Collaborator

example code pls

@bounceme
Copy link
Collaborator

Actually, you are using an older version of the indent script, please install the latest master from this repo

@bounceme
Copy link
Collaborator

or your pr is on master, but the trace shows an older version

@cabaalexander
Copy link

Hello (:

How many tabs and buffers are we talking about? 🤔

@shirohana
Copy link
Author

shirohana commented Jan 19, 2019

Hello (´・ω・`)

It's around 6 tabs and every tab contains at least 4 buffers.

I'll check is it the bug has been fixed later, or I will rebase this pr onto master (๑ơ ω ơ)

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

Successfully merging this pull request may close these issues.

3 participants