NestedTermBuilder class was only making things complicated. Its two use cases, i.e. building symbol and expression designations, were only illusorily similar. In both cases a hash was produced, but its content was almost entirely different.
This pull request simplifies things by replacing the problematic NestedTermBuilder class with two simple topic methods. Additionally, some repeated logic is moved to these methods rather than being evaluated outside of them.
Making this change greatly simplifies upgrading Glossarist dependency to higher versions.
NestedTermBuilderclass was only making things complicated. Its two use cases, i.e. building symbol and expression designations, were only illusorily similar. In both cases a hash was produced, but its content was almost entirely different.This pull request simplifies things by replacing the problematic
NestedTermBuilderclass with two simple topic methods. Additionally, some repeated logic is moved to these methods rather than being evaluated outside of them.Making this change greatly simplifies upgrading Glossarist dependency to higher versions.