Skip to content

bpo-29645: Speed up importing the webbrowser module.#484

Merged
serhiy-storchaka merged 5 commits into
python:masterfrom
serhiy-storchaka:bpo-29645-webbrowser-delayed-initialization
Mar 8, 2017
Merged

bpo-29645: Speed up importing the webbrowser module.#484
serhiy-storchaka merged 5 commits into
python:masterfrom
serhiy-storchaka:bpo-29645-webbrowser-delayed-initialization

Conversation

@serhiy-storchaka

Copy link
Copy Markdown
Member

webbrowser.register() is now thread-safe.

webbrowser.register() is now thread-safe.
@serhiy-storchaka serhiy-storchaka added the type-feature A feature request or enhancement label Mar 5, 2017
@mention-bot

Copy link
Copy Markdown

@serhiy-storchaka, thanks for your PR! By analyzing the history of the files in this pull request, we identified @eric-s-raymond, @freddrake, @rhettinger, @birkenfeld and @tim-one to be potential reviewers.

@serhiy-storchaka

Copy link
Copy Markdown
Member Author

Tests were failed because webbrowser didn't found any browser when DISPLAY and TERM are not set.

@serhiy-storchaka serhiy-storchaka merged commit a7cba27 into python:master Mar 8, 2017
@serhiy-storchaka serhiy-storchaka deleted the bpo-29645-webbrowser-delayed-initialization branch March 8, 2017 15:16
@davesteele

Copy link
Copy Markdown
Contributor

Note that the effects of the register_standard_browser call in register() are blocked by the lock. try_order will eventually get flushed out with the get/open() call, but the pre-registered browser will always be at the start of the list, instead of respecting 'preferred'.

@serhiy-storchaka

Copy link
Copy Markdown
Member Author

Is there something wrong with the code? Please open an issue on the bag tracker.

jaraco pushed a commit that referenced this pull request Dec 2, 2022
* Update runtime to the latest Python3.9 release

* Update runtime.txt

Co-authored-by: Mariatta Wijaya <Mariatta@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type-feature A feature request or enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants