Refactor the Trie container to not need an EndSymbol

[MakoEnergy]

Currently in the Trie implementation we have we cannot accept an iterator pair. The api simply doesn't support that. Instead we pass in a pointer with the expectation that it will be terminated by a user defined end symbol. This is a C-style approach, not C++.

It would take reworking a number of the methods to be template functions but they should be rewritten to handle iterator pairs being passed into them rather than c-style arrays. The iterator pair defines the bounds, rather than a terminator. This affords us better optimizations and flexibility when performing checks against the Trie.