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

Corrected code example in coded_stream.h #73

Merged
merged 1 commit into from Nov 5, 2014

Conversation

acidtonic
Copy link
Contributor

There is an example at the top of the page which contains code that appears to show a user how to create and serialize a message to a file. However the flags to open the file lack the O_CREAT flag which allows creating the file if it doesn't exist. I was troubleshooting a situation where this snippet was used and compiled, but never created a file.

There is an example at the top of the page which contains code that appears to show a user how to create and serialize a message to a file. However the flags to open the file lack the O_CREAT flag which allows creating the file if it doesn't exist. I was troubleshooting a situation where this snippet was used and compiled, but never created a file.
@xfxyjwf
Copy link
Contributor

xfxyjwf commented Nov 4, 2014

Thanks for the patch. Could you sign this Google CLA so we can accept the patch?
https://cla.developers.google.com/about/google-individual

@acidtonic
Copy link
Contributor Author

done. Thanks

@xfxyjwf
Copy link
Contributor

xfxyjwf commented Nov 4, 2014

What name did you use to sign the CLA? I couldn't find your github name among google CLA signers.

@acidtonic
Copy link
Contributor Author

Z Davis. (listed acidtonic@gmail as my email)

@xfxyjwf
Copy link
Contributor

xfxyjwf commented Nov 5, 2014

Found it. Thanks.

@xfxyjwf xfxyjwf merged commit 2ba455f into protocolbuffers:master Nov 5, 2014
TeBoring pushed a commit to TeBoring/protobuf that referenced this pull request Jan 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants