Inverse context creation algorithm: could steps 3.16) and 3.17) be merged?

[ghost]

Hello,

In the inverse context creation algorithm (4.3.2), steps 3.16) and 3.17) look suspiciously similar. Could they be merged? For instance:

3.16.1) Create new variable default-lang-dir.
3.16.2) If active context has a default base direction, set default-lang-dir to the concatenation of default-language and default base direction, separated by '_'.
3.16.3) Otherwise, set default-lang-dir to default-language.
3.16.4) If language map does not have a default-lang-dir entry, create one and set its value to the term being processed.
3.16.5) (verbatim old 3.16.3)
3.16.6) (verbatim old 3.16.4)

Remove 3.17).

What do you think?

Best regards,

Vivien

[pchampin]

Hi Vivien,

that's a good point. In fact, we could even move part of this between steps 2 and 3. It is a bit strange to handle the active context's default language at the top, and the default base direction at the bottom...