Skip to content

perf: Call update_env only once per Markdown instance#201

Merged
pawamoy merged 1 commit intomkdocstrings:masterfrom
oprypin:updenv
Dec 28, 2020
Merged

perf: Call update_env only once per Markdown instance#201
pawamoy merged 1 commit intomkdocstrings:masterfrom
oprypin:updenv

Conversation

@oprypin
Copy link
Copy Markdown
Member

@oprypin oprypin commented Dec 26, 2020

The follow-up calls for each block make no difference at all, and are getting quite expensive.

The follow-up calls for each block make no difference at all, and are getting quite expensive.
@pawamoy pawamoy merged commit 8c16159 into mkdocstrings:master Dec 28, 2020
@pawamoy
Copy link
Copy Markdown
Member

pawamoy commented Dec 28, 2020

Easy change for improved perfs, merging! Thanks @oprypin!

jdpatt added a commit to jdpatt/mkdocstrings that referenced this pull request Jan 6, 2023
Instead of a boolean of has any env been updated, track by handle name
which env has been updated.

References: mkdocstrings#201
Fixes: 502
jdpatt added a commit to jdpatt/mkdocstrings that referenced this pull request Jan 9, 2023
Instead of a boolean of has any env been updated, track by handle name
which env has been updated.

References: mkdocstrings#201
Fixes: 502
pawamoy pushed a commit that referenced this pull request Jan 11, 2023
Instead of using a single boolean,
record which handler's env has been update with a set.

PR #201: #201
Issue #502: #502
PR #507: #507
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