Skip to content

Conversation

itchyny
Copy link
Contributor

@itchyny itchyny commented Jan 20, 2020

I'm not sure which situation these lines are required, but the code was introduced on June 26, 2019 (#395), and lightline fixes for popup on July 15, 2019 (itchyny/lightline.vim#371) so I believe these aren't required anymore.
Ping to @jerdna-regeiz.

@itchyny
Copy link
Contributor Author

itchyny commented Jan 20, 2020

The problem is that when some exception was thrown while setting the filetype, lightline can't be enabled (itchyny/lightline.vim#409 (comment)). One solution is to wrap the setlocal filetype by try catch, and another solution is to stop toggling lightline.

@itchyny itchyny force-pushed the remove-lightline-toggle branch from 3b96fc8 to 605abdf Compare February 17, 2020 02:47
@jerdna-regeiz
Copy link
Contributor

Yes of course, if lightline now is fixed now, just remove the code! (As it already was mentioned in the code comment :) )

@mattn mattn requested review from thomasfaingnaert and mattn and removed request for thomasfaingnaert February 19, 2020 09:51
@mattn mattn merged commit b399cde into prabirshrestha:master Feb 19, 2020
@mattn
Copy link
Collaborator

mattn commented Feb 19, 2020

Thanks all!

@itchyny itchyny deleted the remove-lightline-toggle branch February 19, 2020 10:05
kg8m pushed a commit to kg8m/vim-lsp that referenced this pull request Feb 20, 2020
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