Commit ba31d662 authored by Russ Cox's avatar Russ Cox

codereview: die if initialized twice

If this happens, something is misconfigured.
If we don't test for this explicitly, MatchAt ends
up calling itself recursively forever.

R=golang-dev, r
CC=golang-dev
https://golang.org/cl/5576066
parent 79dc3441
......@@ -2171,10 +2171,21 @@ def norollback(*pats, **opts):
"""(disabled when using this extension)"""
raise hg_util.Abort("codereview extension enabled; use undo instead of rollback")
codereview_init = False
def reposetup(ui, repo):
global codereview_disabled
global defaultcc
global codereview_init
if codereview_init:
raise hg_util.Abort("codereview extension initialized twice")
codereview_init = True
remote = ui.config("paths", "default", "")
if remote.find("://") < 0:
raise hg_util.Abort("codereview: default path '%s' is not a URL" % (remote,))
# Read repository-specific options from lib/codereview/codereview.cfg or codereview.cfg.
root = ''
try:
......
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