From fabff52be59a1114a8a5c83074b0293aefff536e Mon Sep 17 00:00:00 2001 From: Hope Casey-Allen Date: Tue, 30 Oct 2018 12:17:56 -0700 Subject: [PATCH 1/2] Move fallthrough comment so it is recognized internally --- test/core/memory_usage/server.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/core/memory_usage/server.cc b/test/core/memory_usage/server.cc index 3e7bb7e11fe..6f8a9bc0d46 100644 --- a/test/core/memory_usage/server.cc +++ b/test/core/memory_usage/server.cc @@ -292,9 +292,9 @@ int main(int argc, char** argv) { send_status(&calls[k]); } } + /* fallthrough */ // no break here since we want to continue to case // FLING_SERVER_SEND_STATUS_SNAPSHOT to destroy the snapshot call - /* fallthrough */ case FLING_SERVER_SEND_STATUS_SNAPSHOT: grpc_byte_buffer_destroy(payload_buffer); grpc_byte_buffer_destroy(terminal_buffer); From 025bda7c330c6ea9fb4abc8646e1bd8f0bb1a9c7 Mon Sep 17 00:00:00 2001 From: Hope Casey-Allen Date: Tue, 30 Oct 2018 12:55:51 -0700 Subject: [PATCH 2/2] Remove unused BUILD dependencies --- test/core/memory_usage/BUILD | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/test/core/memory_usage/BUILD b/test/core/memory_usage/BUILD index 2f4e60d1773..ab81103224b 100644 --- a/test/core/memory_usage/BUILD +++ b/test/core/memory_usage/BUILD @@ -12,14 +12,12 @@ # See the License for the specific language governing permissions and # limitations under the License. -load("//bazel:grpc_build_system.bzl", "grpc_cc_library", "grpc_cc_test", "grpc_cc_binary", "grpc_package") +load("//bazel:grpc_build_system.bzl", "grpc_cc_test", "grpc_cc_binary", "grpc_package") grpc_package(name = "test/core/memory_usage") licenses(["notice"]) # Apache v2 -load("//test/core/util:grpc_fuzzer.bzl", "grpc_fuzzer") - grpc_cc_binary( name = "client", testonly = 1,