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

parse with length option #659

Closed
douxing opened this issue Jun 14, 2016 · 4 comments
Closed

parse with length option #659

douxing opened this issue Jun 14, 2016 · 4 comments

Comments

@douxing
Copy link

douxing commented Jun 14, 2016

I think that GenericDocument::Parse should have a version with length option, like:

GenericDocument::Parse(const Ch* str, size_t length)

it might be redundant but is very handy with communication data, thus i don't have to reallocate memory for the \0 at the tail after receiving the data.

@gggin
Copy link

gggin commented Jun 14, 2016

you can use std::string version

@pah
Copy link
Contributor

pah commented Jun 14, 2016

This is available in the current master since the merge of #553, see also #158 for more discussion.

@miloyip
Copy link
Collaborator

miloyip commented Jun 15, 2016

Thanks @pah

@miloyip miloyip closed this as completed Jun 15, 2016
@douxing
Copy link
Author

douxing commented Jun 15, 2016

@pah @miloyip after a git pull, every thing is okay, thank you!

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

No branches or pull requests

4 participants