AJ Heller
fc7314c701
Avoid fully qualifying namespaces (and add check) ( #28901 )
...
Based on a handful of https://abseil.io/tips , it's generally advised to
only fully-qualify namespaces when in a `using` statement, or when it's
otherwise required for compilation. In all other cases, the general
recommendation is to not fully-qualify.
This change fixes most `grpc.*` namespace uses. There are potential
challenges in trying to make blanket changes to non-gRPC namespace uses,
such as `::testing`, since there is also a `grpc::testing` namespace.
3 years ago
Craig Tiller
28316341bf
Annotate impl/codegen with IWYU pragmas ( #27252 )
...
* Add a tool to annotate impl/codegen with IWYU pragmas
* xx
* oops
* fmt
* x
* fix wrong direction bug
* use defaultdict
* better annotations
* better annotations
* Automated change: Fix sanity tests
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
3 years ago
AJ Heller
0407f4be49
Fix free-after-use bug in server callback handler ( #27187 )
...
Successful asan run https://source.cloud.google.com/results/invocations/33bc587c-a440-45bd-9886-a23789645a7d/targets
Successful TGP available upon request.
3 years ago
Vijay Pai
cf1ebf7387
Remove experimental tag from uses of the C++ callback API ( #26495 )
3 years ago
Vijay Pai
0f80378a21
Resolve race between OnReadDone(ok=false) and IsCancelled ( #26245 )
...
* Resolve and test race between OnReadDone(ok=false) and IsCancelled
* Fix retry case
* Fix health check case
* Address reviewer comments.
* ADD TODO requested by markdroth
4 years ago
ZhouyihaiDing
a584bc4f02
Support ServerContext for callback API
4 years ago
Vijay Pai
91bb4e178c
Check for nullptr (deser failure) before destructing included request
4 years ago
Esun Kim
decc199ca8
Fix by clang-tidy
4 years ago
Esun Kim
34789b2d83
Fix by clang-format-8.
4 years ago
Alisha Nanda
25c35b6db7
Revert "Revert "Merge pull request #24047 from grpc/revert-23916-server-context""
...
This reverts commit 84c4994dc7
.
4 years ago
Alisha Nanda
84c4994dc7
Revert "Merge pull request #24047 from grpc/revert-23916-server-context"
...
This reverts commit 7c098321bd
, reversing
changes made to d109f4b406
.
4 years ago
Karthik Ravi Shankar
b657c8d79c
Revert "Revert "Move ServerContext from ::grpc_impl to ::grpc""
4 years ago
Karthik Ravi Shankar
f1abdd1725
Revert "Move ServerContext from ::grpc_impl to ::grpc"
4 years ago
Karthik Ravi Shankar
a23578444c
Fix and format build after rebase
4 years ago
Karthik Ravi Shankar
56d7d9e804
Move ServerContext from ::grpc_impl to ::grpc
4 years ago
Vijay Pai
2b0f14d54d
Don't drop message on WriteAndFinish with non-ok status
4 years ago
Vijay Pai
3620abf059
Properly follow precise conditions for callback inlining
5 years ago
Vijay Pai
94a8c5888a
Remove namespace experimental from namespace grpc_impl
5 years ago
Vijay Pai
be6979202b
Revamp server-side C++ callback API, message allocator, and mocking
5 years ago