Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(6115)

Issue 122960043: code review 122960043: encoding/xml: add InputOffset method to Decoder (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
9 years, 8 months ago by rsc
Modified:
9 years, 8 months ago
Reviewers:
r, gobot, dave, bradfitz
CC:
bradfitz, golang-codereviews
Visibility:
Public.

Description

encoding/xml: add InputOffset method to Decoder Among other things, this allows users to match the decoded pieces with the original XML, which can be necessary for implementing standards like XML signatures. Fixes issue 8484.

Patch Set 1 #

Patch Set 2 : diff -r 88999b670ed9 https://code.google.com/p/go/ #

Patch Set 3 : diff -r 88999b670ed9 https://code.google.com/p/go/ #

Patch Set 4 : diff -r 88999b670ed9 https://code.google.com/p/go/ #

Patch Set 5 : diff -r 70a668a2e09e1587fee166f92c265861c57a490f https://code.google.com/p/go/ #

Patch Set 6 : diff -r db85c522372c38d9fbae9f80309da5399cee8578 https://code.google.com/p/go/ #

Unified diffs Side-by-side diffs Delta from patch set Stats (+38 lines, -4 lines) Patch
M src/pkg/encoding/xml/xml.go View 1 2 3 4 chunks +11 lines, -0 lines 0 comments Download
M src/pkg/encoding/xml/xml_test.go View 1 2 3 4 5 chunks +27 lines, -4 lines 0 comments Download

Messages

Total messages: 8
bradfitz
LGTM test?
9 years, 8 months ago (2014-08-06 16:53:13 UTC) #1
rsc
I'll add a test before I send it for review. :-)
9 years, 8 months ago (2014-08-06 18:27:09 UTC) #2
rsc
Hello bradfitz@golang.org (cc: golang-codereviews@googlegroups.com), I'd like you to review this change to https://code.google.com/p/go/
9 years, 8 months ago (2014-08-06 21:05:10 UTC) #3
bradfitz
LGTM
9 years, 8 months ago (2014-08-06 21:10:10 UTC) #4
rsc
*** Submitted as https://code.google.com/p/go/source/detail?r=18c464e4811e *** encoding/xml: add InputOffset method to Decoder Among other things, this ...
9 years, 8 months ago (2014-08-06 22:00:10 UTC) #5
gobot
This CL appears to have broken the nacl-arm builder. See http://build.golang.org/log/6dac54b5cb4d1abac23959e70e4bc050d477d1ec
9 years, 8 months ago (2014-08-06 22:39:01 UTC) #6
dave_cheney.net
I'm guessing these exit 255 errors are the build being cancelled manually? On 7 Aug ...
9 years, 8 months ago (2014-08-06 22:54:08 UTC) #7
r
9 years, 8 months ago (2014-08-06 23:33:52 UTC) #8
Message was sent while issue was closed.
go1.4.txt please
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b