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

[c++ grpc] Fix includes when bond::Void is used #738

Merged

Conversation

chwarr
Copy link
Member

@chwarr chwarr commented Nov 29, 2017

When a service contains events or parameterless methods, the generated
code uses bond::Void as the request/result type. To serialize
bond::Void, bond_reflection.h needs to be included.

The generated service code now includes bond_reflection.h if any service
contains an event or a parameterless method.

Fixes #735

When a service contains events or parameterless methods, the generated
code uses bond::Void as the request/result type. To serialize
bond::Void, bond_reflection.h needs to be included.

The generated service code now includes bond_reflection.h if any service
contains an event or a parameterless method.

Fixes microsoft#735
@ara-ayvazyan
Copy link
Contributor

ara-ayvazyan commented Nov 29, 2017

@chwarr, is it possible to use gRPC with bond_apply?

@chwarr
Copy link
Member Author

chwarr commented Dec 5, 2017

It should work with bond_apply.lib; however, you'd have to make sure to #include <bond/core/bond_apply.h> before any of the generated _grpc.h headers. We have an example that does this for the user-generated type.

@ara-ayvazyan ara-ayvazyan merged commit 95c4f9d into microsoft:master Dec 6, 2017
@chwarr chwarr deleted the chwarr/grpc-include-reflection branch December 7, 2017 00:23
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.

3 participants