Recursion in Equivset mappings
Closed, InvalidPublic
Actions

Assigned To

None

Authored By

	Huji
	Feb 10 2020, 2:16 PM

Description

Equivset replaces | (the pipe character, unicode 7C) with 1 but then later on replaces ǀ (unicode 1C0) with the pipe character.

Both should have the same output, i.e. 1.

The current code can have negative impact when the output is being used as part of a regexp pattern (see T244746 for instance).

	Subject	Repo	Branch	Lines +/-
	Remove recursion in Equiset mappings	mediawiki/libs/Equivset	master	+1 -1

Mentioned Here: T244746: Make it easier to generate normalized regex patterns

Change 571289 had a related patch set uploaded (by Huji; owner: Huji):
[mediawiki/libs/Equivset@master] Remove recursion in Equiset mappings

On second thought, this is not necessary. Equivset already walks through the recursions before generating the final dist mapping.

Change 571289 abandoned by Huji:
Remove recursion in Equiset mappings

Reason:
Not needed.