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
json: Add InputOffset for stream byte offset access #29686
Conversation
This PR (HEAD: b046645) has been imported to Gerrit for code review. Please visit https://go-review.googlesource.com/c/go/+/157519 to see it. Tip: You can toggle comments from me using the |
Message from Gobot Gobot: Patch Set 1: Congratulations on opening your first change. Thank you for your contribution! Next steps: Most changes in the Go project go through a few rounds of revision. This can be During May-July and Nov-Jan the Go project is in a code freeze, during which Please don’t reply on this GitHub thread. Visit golang.org/cl/157519. |
Message from Brad Fitzpatrick: Patch Set 1: (1 comment) I'll mark this CL on hold until the proposal is approved. Please don’t reply on this GitHub thread. Visit golang.org/cl/157519. |
Message from Brad Fitzpatrick: Patch Set 1: Please reference the proposal GitHub issue from this commit message. Please don’t reply on this GitHub thread. Visit golang.org/cl/157519. |
Message from Daniel Martí: Patch Set 2: The proposal was accepted. Adam, can you update the CL as per Brad's comments and rebase? Also, please follow the commit message best practices. Please don’t reply on this GitHub thread. Visit golang.org/cl/157519. |
Message from Ian Lance Taylor: Patch Set 3: Commit message was updated. Please don’t reply on this GitHub thread. Visit golang.org/cl/157519. |
Message from Ian Lance Taylor: Patch Set 4: Patch Set 3 was rebased Please don’t reply on this GitHub thread. Visit golang.org/cl/157519. |
Message from Ian Lance Taylor: Patch Set 4: Run-TryBot+1 A new CL https://golang.org/cl/200677 has been sent. Adam, do you want to update the Please don’t reply on this GitHub thread. Visit golang.org/cl/157519. |
Message from Gobot Gobot: Patch Set 4: TryBots beginning. Status page: https://farmer.golang.org/try?commit=510ac3ef Please don’t reply on this GitHub thread. Visit golang.org/cl/157519. |
Message from Gobot Gobot: Patch Set 4: Build is still in progress... Other builds still in progress; subsequent failure notices suppressed until final report. Consult https://build.golang.org/ to see whether they are new failures. Keep in mind that TryBots currently test exactly your git commit, without rebasing. If your commit's git parent is old, the failure might've already been fixed. Please don’t reply on this GitHub thread. Visit golang.org/cl/157519. |
Message from Ian Lance Taylor: Patch Set 5: OK, I guess I can't change the commit message, as Gerritbot will just change it back. Please don’t reply on this GitHub thread. Visit golang.org/cl/157519. |
This PR is being closed because golang.org/cl/157519 has been abandoned. Replaced by CL 200677. |
I'm sending proposal along with pull request, please don't feel I'm trying to force anything.
Use case: I'm trying to index entries in JSON file by their byte offset position for random access
This pull request makes
offset
method public and changes its name toInputOffset
to match XML's decoder API: https://golang.org/pkg/encoding/xml/#Decoder.InputOffsetReference to proposal: #29688