From: David Briscoe <idbrii@gmail.com>
Date: Fri, 7 Dec 2018 00:45:35 +0000 (-0800)
Subject: Prevent repeated sourcing of syntax
X-Git-Url: https://git.madduck.net/etc/vim.git/commitdiff_plain/da145debf80da5f38e244d2f968d43f5051590f1?ds=sidebyside;hp=52ee2eb68a706972a1840ca036035033046568d6

Prevent repeated sourcing of syntax

Bug: Using `vim --startuptime ~/time.log name.md` would show multiple
sourcing of syntax/markdown.vim indicating unnecessary startup cost.

Cause: As of vim/vim@7d76c804af900ba6dcc4b1e45373ccab3418c6b2, vim uses
.md for markdown instead of modula2. Since vim-markdown was using `set
filetype`, it was sourcing the files a second time after vim's runtime
files had already set the filetype.

Resolution: Using `setfiletype` prevents this double sourcing. To remain
backwards compatible, we remove the .md filetype detection for modula2
on old versions.

Related to sheerun/vim-polyglot#290.

Tested .md ftdetection with vim 7.4.052 and 8.1.527.
---

diff --git a/ftdetect/markdown.vim b/ftdetect/markdown.vim
index 611ecaa..07682da 100644
--- a/ftdetect/markdown.vim
+++ b/ftdetect/markdown.vim
@@ -1,3 +1,8 @@
+if !has('patch-7.4.480')
+    " Before this patch, vim used modula2 for .md.
+    au! filetypedetect BufRead,BufNewFile *.md
+endif
+
 " markdown filetype file
-au BufRead,BufNewFile *.{md,mdown,mkd,mkdn,markdown,mdwn} set filetype=markdown
-au BufRead,BufNewFile *.{md,mdown,mkd,mkdn,markdown,mdwn}.{des3,des,bf,bfa,aes,idea,cast,rc2,rc4,rc5,desx} set filetype=markdown
+au BufRead,BufNewFile *.{md,mdown,mkd,mkdn,markdown,mdwn} setfiletype markdown
+au BufRead,BufNewFile *.{md,mdown,mkd,mkdn,markdown,mdwn}.{des3,des,bf,bfa,aes,idea,cast,rc2,rc4,rc5,desx} setfiletype markdown