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

Update BUILD for the recent json util change. #509

Merged
merged 1 commit into from
Jun 18, 2015

Conversation

liujisi
Copy link
Contributor

@liujisi liujisi commented Jun 18, 2015

@xfxyjwf for review.

Change-Id: I0b5a2c017f5412aaca3a47815671753cdef63165
@xfxyjwf
Copy link
Contributor

xfxyjwf commented Jun 18, 2015

LGTM

liujisi added a commit that referenced this pull request Jun 18, 2015
Update BUILD for the recent json util change.
@liujisi liujisi merged commit 89834be into protocolbuffers:master Jun 18, 2015
@liujisi liujisi deleted the update_BUILD branch August 3, 2017 20:46
taoso pushed a commit to taoso/protobuf that referenced this pull request Aug 1, 2018
As mentioned in protocolbuffers#509, jsonpb panics when marshaling/unmarshaling a non-generated message that has an unexported pointer field. This change will restore dynamic messages to working with jsonpb (they work fine in master; just broken in this dev branch as of protocolbuffers#472).
adellahlou pushed a commit to adellahlou/protobuf that referenced this pull request Apr 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants