search

ternjs / tern_for_sublime

Sublime Text package adding Tern support
MIT License
803 stars 54 forks source link

View size over 8000 #11

Closed subhaze closed 11 years ago

subhaze commented 11 years ago

From what I can tell the view size can be over 8000 and edits can be made quickly and easily, but, when you get down to the end of the view and start editing on lines of code that are over the 8000 mark Sublime Text just locks up.

https://github.com/marijnh/tern_for_sublime/blob/master/tern.py#L252

I was just now able to test on Sublime Text 2 and it's fine there, so this seems to only effect Sublime Text 3.

subhaze commented 11 years ago

I've finally had some time to investigate this further and it's not a bug in the plugin.

In SublimeText 3 view.find(..) incorrectly returns (-1, -1) instead of None.

Debug logs:

(7188, 7196)
(7368, 7376)
(7565, 7573)
(-1, -1)

I've created a patch here https://github.com/subhaze/tern_for_sublime/commit/8c066c67820fd2acaed91f1f306a2ab5b35ef007 for the time being.

The Sublime Text forum seems to be down... So I can't file a bug report... but I'll try to check later today to file one :)

marijnh commented 11 years ago

Thanks for looking into this! I've merged in your hack, so that others won't have to run into the hang.

subhaze commented 11 years ago

No prob man.

I've posted on the forum about this bug: http://www.sublimetext.com/forum/viewtopic.php?f=3&t=12499