Closed DaleFarnsworth closed 6 years ago
That's fine with me.
I will point out that the function is called only a single time in the normal case. After that, a single test of an int value is performed on each lookup. The overhead is inconsequential.
I think this would best be conditionally compiled (e.g. "#ifdef DEBUG" or similar).
That's my plan. Similar to what I just did for the new PTT Last Heard code.
After this commit is applied, if either the state or country abbreviation lists are not in sorted order, both the state and country fields are set to "unsorted abbreviations". I'm not sure it's worth applying.