Skip to content

Remove unused function - #6911

Merged
creamsoup merged 1 commit into
grpc:masterfrom
agasparovic-sabre:patch-1
Apr 8, 2020
Merged

Remove unused function#6911
creamsoup merged 1 commit into
grpc:masterfrom
agasparovic-sabre:patch-1

Conversation

@agasparovic-sabre

Copy link
Copy Markdown
Contributor

INFO: From Compiling external/io_grpc_grpc_java/compiler/src/java_plugin/cpp/java_generator.cpp [for host]:
external/io_grpc_grpc_java/compiler/src/java_plugin/cpp/java_generator.cpp:164:20: warning: unused function 'ShouldGenerateAsLite' [-Wunused-function]
static inline bool ShouldGenerateAsLite(const Descriptor* desc) {
^
1 warning generated.

INFO: From Compiling external/io_grpc_grpc_java/compiler/src/java_plugin/cpp/java_generator.cpp [for host]:
external/io_grpc_grpc_java/compiler/src/java_plugin/cpp/java_generator.cpp:164:20: warning: unused function 'ShouldGenerateAsLite' [-Wunused-function]
static inline bool ShouldGenerateAsLite(const Descriptor* desc) {
                   ^
1 warning generated.
@linux-foundation-easycla

linux-foundation-easycla Bot commented Apr 8, 2020

Copy link
Copy Markdown

CLA Check
The committers are authorized under a signed CLA.

@creamsoup creamsoup added the kokoro:run Add this label to a PR to tell Kokoro the code is safe and tests can be run label Apr 8, 2020
@grpc-kokoro grpc-kokoro removed the kokoro:run Add this label to a PR to tell Kokoro the code is safe and tests can be run label Apr 8, 2020
@dapengzhang0

Copy link
Copy Markdown
Contributor

@carl-mastrangelo

Copy link
Copy Markdown
Contributor

LGTM, Seems, fine, not sure if I have a stake though.

@creamsoup
creamsoup merged commit 2119fac into grpc:master Apr 8, 2020
@creamsoup

Copy link
Copy Markdown
Contributor

merged, thanks @agasparovic-sabre!

dfawley pushed a commit to dfawley/grpc-java that referenced this pull request Jan 15, 2021
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Jun 15, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants