This is an archive of the discontinued Mercurial Phabricator instance.

Status	Author	Revision
Closed	martinvonz	D5352 match: drop unnecessary wrapping of regex in group
Closed	martinvonz	D5351 match: use _BASE_SIZE instead of magic value 4
Closed	martinvonz	D5350 match: make "groupsize" include the trailing "\|"
Closed	martinvonz	D5349 match: fix an unaligned (but harmless) indent

Diff 12667

	allgroups = []			allgroups = []
	regexps = [_regex(k, p, globsuffix) for (k, p, s) in kindpats]			regexps = [_regex(k, p, globsuffix) for (k, p, s) in kindpats]
	fullregexp = _joinregexes(regexps)			fullregexp = _joinregexes(regexps)

	startidx = 0			startidx = 0
	groupsize = _BASE_SIZE			groupsize = _BASE_SIZE
	for idx, r in enumerate(regexps):			for idx, r in enumerate(regexps):
	piecesize = len(r)			piecesize = len(r)
	if (piecesize + 4) > MAX_RE_SIZE:			if (piecesize + _BASE_SIZE) > MAX_RE_SIZE:
	msg = _("matcher pattern is too long (%d bytes)") % piecesize			msg = _("matcher pattern is too long (%d bytes)") % piecesize
	raise error.Abort(msg)			raise error.Abort(msg)
	elif (groupsize + piecesize) > MAX_RE_SIZE:			elif (groupsize + piecesize) > MAX_RE_SIZE:
	group = regexps[startidx:idx]			group = regexps[startidx:idx]
	allgroups.append(_joinregexes(group))			allgroups.append(_joinregexes(group))
	startidx = idx			startidx = idx
	groupsize = _BASE_SIZE			groupsize = _BASE_SIZE
	groupsize += piecesize + 1			groupsize += piecesize + 1