This is an automated email from the ASF dual-hosted git repository.
wwbmmm pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/brpc.git
The following commit(s) were added to refs/heads/master by this push:
new edb9caf7 Fix comparison warning in http2_rpc_protocol.cpp (#2626)
edb9caf7 is described below
commit edb9caf7649ea6dd6e4a7abe9c5c5f132de1ac44
Author: Yu Dou <imdo...@gmail.com>
AuthorDate: Mon May 6 10:30:39 2024 +0800
Fix comparison warning in http2_rpc_protocol.cpp (#2626)
---
src/brpc/policy/http2_rpc_protocol.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/brpc/policy/http2_rpc_protocol.cpp
b/src/brpc/policy/http2_rpc_protocol.cpp
index 5b0538d3..f9b01206 100644
--- a/src/brpc/policy/http2_rpc_protocol.cpp
+++ b/src/brpc/policy/http2_rpc_protocol.cpp
@@ -744,7 +744,7 @@ H2ParseResult H2StreamContext::OnData(
const int64_t acc = _deferred_window_update.fetch_add(frag_size,
butil::memory_order_relaxed) + frag_size;
// Allocate the quota of the window to each stream.
- if (acc >= _conn_ctx->local_settings().stream_window_size /
(_conn_ctx->VolatilePendingStreamSize() + 1)) {
+ if (acc >=
static_cast<int64_t>(_conn_ctx->local_settings().stream_window_size) /
(_conn_ctx->VolatilePendingStreamSize() + 1)) {
if (acc > _conn_ctx->local_settings().stream_window_size) {
LOG(ERROR) << "Fail to satisfy the stream-level flow control
policy";
return MakeH2Error(H2_FLOW_CONTROL_ERROR, frame_head.stream_id);
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@brpc.apache.org
For additional commands, e-mail: dev-h...@brpc.apache.org
