Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

encoding/xml: Expose xml.Decoder.line #43589

Closed
rzajac opened this issue Jan 8, 2021 · 1 comment
Closed

encoding/xml: Expose xml.Decoder.line #43589

rzajac opened this issue Jan 8, 2021 · 1 comment
Labels
FeatureRequest FrozenDueToAge NeedsDecision Feedback is required from experts, contributors, and/or the community before a change can be made.
Milestone

Comments

@rzajac
Copy link

rzajac commented Jan 8, 2021

What version of Go are you using (go version)?

$ go version
go version go1.15.1 linux/amd64

Does this issue reproduce with the latest release?

yes

What did you do?

Creating custom XML schema validator.

What did you expect to see?

It would be great to expose xml.Decoder.line. Creating custom errors with line numbers is better then offset. The offset is already available through InputOffset() method.

@rzajac rzajac changed the title Expose xml.Decoder.line Expose xml.Decoder.line Jan 8, 2021
@ALTree ALTree changed the title Expose xml.Decoder.line encoding/xml: Expose xml.Decoder.line Jan 9, 2021
@ALTree ALTree added FeatureRequest NeedsDecision Feedback is required from experts, contributors, and/or the community before a change can be made. labels Jan 9, 2021
@ALTree ALTree added this to the Unplanned milestone Jan 9, 2021
@seankhliao
Copy link
Member

Merging into proposal #45628

@golang golang locked and limited conversation to collaborators Apr 19, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
FeatureRequest FrozenDueToAge NeedsDecision Feedback is required from experts, contributors, and/or the community before a change can be made.
Projects
None yet
Development

No branches or pull requests

4 participants