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.

Sign up for GitHub

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

Jump to bottom

mini-fix #2989

Closed
wants to merge 9 commits into from
Closed

mini-fix #2989

wants to merge 9 commits into from

Commits on Aug 20, 2024

  1. change perf testing [1022±4 ms] 

    `hyperfine -n 15 "vi -u NONE -c 'source test.vim'"` gives `1.022 ± 0.004 s`
    @ces42
    ces42 committed Aug 20, 2024
    Configuration menu
    Copy the full SHA
    784ae92 View commit details
    Browse the repository at this point in the history

  2. optimize regex for \mathrm, \mathbf etc. [1002±5 ms] 

    `hyperfine -n 15 "vi -u NONE -c 'source test.vim'"` gives `1.002 ± 0.005 s`
    @ces42
    ces42 committed Aug 20, 2024
    Configuration menu
    Copy the full SHA
    11e64a2 View commit details
    Browse the repository at this point in the history

  3. optimize regex for some delimiters [983±8 ms] 

    `hyperfine -n 15 "vi -u NONE -c 'source test.vim'"` gives `982.8 ± 8.2 ms`
    @ces42
    ces42 committed Aug 20, 2024
    Configuration menu
    Copy the full SHA
    69165ca View commit details
    Browse the repository at this point in the history

  4. optimize regex for texCmdStyle and texCmdSize [911±4 ms] 

    `hyperfine -n 15 "vi -u NONE -c 'source test.vim'"` gives `911.0 ± 3.7 ms`
    @ces42
    ces42 committed Aug 20, 2024
    Configuration menu
    Copy the full SHA
    9658d97 View commit details
    Browse the repository at this point in the history

  5. try to only create one big texMathEnvBgnEnd rule [857±6 ms] 

    (also only one `texMathError` rule for un-matched `\end`s)

I had to create a separate group `texMathEnvBgnEnd2` which is a bit
jank.

`hyperfine -n 15 "vi -u NONE -c 'source test.vim'"` gives `856.8 ± 5.7 ms`
    @ces42
    ces42 committed Aug 20, 2024
    Configuration menu
    Copy the full SHA
    1db3cc4 View commit details
    Browse the repository at this point in the history

  6. only create ont texMathZoneEnv rule [823±6 ms] 

    also try a bit harder to only create one big `texMathEnvBgnEnd` rule, and I got rid of `texMathEnvBgnEnd2`.
Unfortunately I had to restrict funcionality a bit, it is no longer
possible to specify predicates when using `new_env` with `{'math':
v:true}`

`hyperfine -n 15 "vi -u NONE -c 'source test.vim'"` gives `822.7 ± 6.3 ms`
    @ces42
    ces42 committed Aug 20, 2024
    Configuration menu
    Copy the full SHA
    0b6dbb7 View commit details
    Browse the repository at this point in the history

  7. further merge texMathDelim rules [808±6 ms]

    @ces42
    ces42 committed Aug 20, 2024
    Configuration menu
    Copy the full SHA
    09db09c View commit details
    Browse the repository at this point in the history

  8. merge texMathDelim rules for \{\} [804±6 ms]

    @ces42
    ces42 committed Aug 20, 2024
    Configuration menu
    Copy the full SHA
    b0af22b View commit details
    Browse the repository at this point in the history

  9. just pick any window for inverse search if there are several

    @ces42
    ces42 committed Aug 20, 2024
    Configuration menu
    Copy the full SHA
    a886799 View commit details
    Browse the repository at this point in the history