Read about our upcoming Code of Conduct on this issue

Commit b2f7647a authored by Bryan O'Sullivan's avatar Bryan O'Sullivan
Browse files

extensions: rename _ignore to _builtin, add descriptive comment

It was previously not at all obvious what this was for.

We also change it to a set to avoid iterating over an admittedly
small list repeatedly at startup time.
parent f04bd381e8c0
......@@ -24,7 +24,8 @@ from . import (
_extensions = {}
_aftercallbacks = {}
_order = []
_ignore = ['hbisect', 'bookmarks', 'parentrevspec', 'interhg', 'inotify']
# former extensions now in core - we ignore these if found in hgrc
_builtin = set(['hbisect', 'bookmarks', 'parentrevspec', 'interhg', 'inotify'])
def extensions(ui=None):
if ui:
......@@ -75,7 +76,7 @@ def load(ui, name, path):
shortname = name[6:]
shortname = name
if shortname in _ignore:
if shortname in _builtin:
return None
if shortname in _extensions:
return _extensions[shortname]
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment