Skip to content

remove unused variables#16933

Merged
alyssawilk merged 1 commit intoenvoyproxy:mainfrom
giantcroc:main
Jun 21, 2021
Merged

remove unused variables#16933
alyssawilk merged 1 commit intoenvoyproxy:mainfrom
giantcroc:main

Conversation

@giantcroc
Copy link
Copy Markdown

Signed-off-by: giantcroc 1204449533@qq.com

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Deprecated:]
[Optional API Considerations:]

@repokitteh-read-only
Copy link
Copy Markdown

Hi @giantcroc, welcome and thank you for your contribution.

We will try to review your Pull Request as quickly as possible.

In the meantime, please take a look at the contribution guidelines if you have not done so already.

🐱

Caused by: #16933 was opened by giantcroc.

see: more, trace.

@daixiang0
Copy link
Copy Markdown
Member

lgtm

/cc @phlax need start a clean process for this case? I guess there are some -Wunused-but-set-parameter warnings left.

@daixiang0
Copy link
Copy Markdown
Member

/retest

@repokitteh-read-only
Copy link
Copy Markdown

Retrying Azure Pipelines:
Check envoy-presubmit isn't fully completed, but will still attempt retrying.
Retried failed jobs in: envoy-presubmit

🐱

Caused by: a #16933 (comment) was created by @daixiang0.

see: more, trace.

@giantcroc
Copy link
Copy Markdown
Author

/retest

@repokitteh-read-only
Copy link
Copy Markdown

Retrying Azure Pipelines:
Check envoy-presubmit isn't fully completed, but will still attempt retrying.
Retried failed jobs in: envoy-presubmit

🐱

Caused by: a #16933 (comment) was created by @giantcroc.

see: more, trace.

Signed-off-by: giantcroc <1204449533@qq.com>
@zuercher
Copy link
Copy Markdown
Member

/retest

@repokitteh-read-only
Copy link
Copy Markdown

Retrying Azure Pipelines:
Retried failed jobs in: envoy-presubmit

🐱

Caused by: a #16933 (comment) was created by @zuercher.

see: more, trace.

@alyssawilk alyssawilk merged commit ed81f0c into envoyproxy:main Jun 21, 2021
leyao-daily pushed a commit to leyao-daily/envoy that referenced this pull request Sep 30, 2021
Signed-off-by: giantcroc <1204449533@qq.com>
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.

5 participants