Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added no-op for variable to avoid a warning under Linux #574

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

tachoknight
Copy link
Contributor

Using Clang 13, the switches -Werror -Wunused-but-set-variable causes an error because the to variable is set, but under Linux it is never used, causing the build to fail. This PR performs a no-op on the variable in the linux block to avoid this issue.

@shahmishal
Copy link
Member

cc: @rokhinip

@rokhinip rokhinip self-requested a review September 17, 2021 18:29
@shahmishal
Copy link
Member

@swift-ci test

@tomerd
Copy link

tomerd commented Sep 17, 2021

cc @compnerd

@tomerd
Copy link

tomerd commented Sep 17, 2021

seems fine to me

@shahmishal
Copy link
Member

@swift-ci test

@AreaZR
Copy link
Contributor

AreaZR commented Sep 28, 2022

Any updates on this?

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