segments_by_name() renamed to segments_by_identifier()... Naming is hard, and the spec uses the word identifier and segment ID so gaining consistency there.
Do we want to mark the old name as deprecated and point top the new function, or is it not actually being used in the wild?
Field::value() marked as deprecated as a full dupe of Field::as_str()
As part of the information hiding I've removed public visibility from some struct members, let me know if you were using the direct access for anything
A couple of public API changes in here:
segments_by_name()renamed tosegments_by_identifier()... Naming is hard, and the spec uses the wordidentifierandsegment IDso gaining consistency there.deprecatedand point top the new function, or is it not actually being used in the wild?Field::value()marked as deprecated as a full dupe ofField::as_str()