Skip to content

py: use pre-compiled regexps#88

Open
blueyed wants to merge 1 commit into
vim-jp:masterfrom
blueyed:py-performance
Open

py: use pre-compiled regexps#88
blueyed wants to merge 1 commit into
vim-jp:masterfrom
blueyed:py-performance

Conversation

@blueyed

@blueyed blueyed commented Aug 29, 2018

Copy link
Copy Markdown
Contributor

Before:

% time python py/vimlparser.py autoload/vimlparser.vim |md5sum                                                                                                          ⟳(c) ⟳(m) 🐍?
861997350d8e69dc32937d77afb5b931  -
python py/vimlparser.py autoload/vimlparser.vim  3.21s user 0.07s system 99% cpu 3.282 total
md5sum  0.00s user 0.00s system 0% cpu 3.282 total

After:

% time python py/vimlparser.py autoload/vimlparser.vim |md5sum                                                                                                         🐍?
861997350d8e69dc32937d77afb5b931  -
python py/vimlparser.py autoload/vimlparser.vim  2.94s user 0.04s system 99% cpu 2.979 total
md5sum  0.00s user 0.00s system 0% cpu 2.979 total

@tyru

tyru commented Sep 15, 2018

Copy link
Copy Markdown
Member

This seems to be on the right track for me (except failing tests).
@vim-jp/vim-vimlparser what do you think about this?

@tyru

tyru commented Mar 6, 2020

Copy link
Copy Markdown
Member

@blueyed Is this PR necessary even after #142 ?

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.

2 participants