proposal: encoding/xml: Collect metadata, like order and line numbers when parsing XML about go

[ianlancetaylor]

Please describe the new API you are suggesting. Thanks.

[cague]

There isn't a new API in the sense that there are new functions. There are new "field tags" for structures.

xml:",order" // Stores the order of the Element within the Element's parent Element
xml:",line" // Stores the line number for the start of the Element

So when Go's regular Unmarshal function is used, the Go code will fill in those values. See ProblemsXML structure in the other comments to see the field tags in use.

[ianlancetaylor]

OK, can you write out the new documentation that would be added to the encoding/xml package? Thanks.

[cague]

This is the current doc here for Unmarshal
https://pkg.go.dev/encoding/xml#Unmarshal
There is a list of bullet points describing struct field tags.
Here's one of the existing bullet points:

• If the XML element contains a sub-element that hasn't matched any of the above rules and the struct has a field with tag ",any", unmarshal maps the sub-element to that struct field.

The new doc would add more bullet points:

• A struct field with tag ",line" contains the line number for the start Element
• A struct field with tag ",order" contains the order of the Element within the Element's parent Element

And then maybe also have a field tag for catching "out of order" elements.

• A struct field with tag ",ooorder" contains information on Elements that do not follow the order of the structure defined and passed to Unmarshal. The struct field must be of type OutOfOrderItems

type OutOfOrderItems []OutOfOrderItem

type OutOfOrderItem struct {
ElementOOO string // Out of order child element that appears after ElementMarker in the parsed XML
ElementMarker string // The element that appears before ElementOOO in the parsed XML but is defined after ElementOOO in the structure that Unmarshal stores the parsed results
}

This is similar to XML schema element xs:sequence and can be used to help enforce order.
e.g.
msg = fmt.Sprintf("out of order: Element "%v" must be before Element "%v"", oooi.ElementOOO , oooi.ElementMarker)

[ianlancetaylor]

Thanks.