We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
We can rewrite the serialization and deserialization for send/recv using the following sample code, thanks Xu laoshi for reminding.
sendrecv::VariableMessage request; request.set_name(var_name); ... std::string header; request.AppendToString(&header); ProtoEncodeHelper e(static_cast<char*>(buf), 1024); e.WriteRaw(header.data(), header.size())
The text was updated successfully, but these errors were encountered:
@gongweibao is there a way to simplify variable_response.cc code?
variable_response.cc
Sorry, something went wrong.
Deserialization is implemented in variable_reponse.cc. Do you mean to merge them into ProtoEncodeHelper class?
ProtoEncodeHelper
Please check out #10531, in which I simplified serialization codes, can we do the same thing in variable_reponse.cc?
variable_reponse.cc
It's a good idea.
We can't deserialize part of proto. So I close this issue.
gongweibao
typhoonzero
No branches or pull requests
We can rewrite the serialization and deserialization for send/recv using the following sample code, thanks Xu laoshi for reminding.
The text was updated successfully, but these errors were encountered: