Skip to content

[stdlib][Qol] SR-11295 Removing stdlib unnecessary coercions #27165

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

Merged

Conversation

LucianoPAlmeida
Copy link
Contributor

This removes unnecessary 'as' same type casts.
Extension of #26710, just separating this in other PR.

cc @xedin

@xwu xwu changed the title [stdlib][Qol] SR-11295 Removing stdlib uncessary coercions [stdlib][Qol] SR-11295 Removing stdlib unnecessary coercions Sep 14, 2019
Copy link
Member

@natecook1000 natecook1000 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, @LucianoPAlmeida! One small nit, otherwise looks good 👍

@LucianoPAlmeida
Copy link
Contributor Author

Just updating the initial description
The original diagnostic PR was reverted because of an issue with type variables and contextual types, so it is being re-applied on #27895 :)

@CodaFi
Copy link
Contributor

CodaFi commented Dec 4, 2019

@swift-ci please smoke test

@LucianoPAlmeida
Copy link
Contributor Author

Thank's @CodaFi :)
The linux failure seems unrelated

@LucianoPAlmeida
Copy link
Contributor Author

Although the re-apply of the diagnostic is still in the workings, this is actually not dependent on it. And since it is only very simple removals, I was wondering if we get this in if everything is ok?
Also, can someone with rights kick the CI? if everything is ok of course :)

@CodaFi
Copy link
Contributor

CodaFi commented Dec 11, 2019

Sure. In the future, though, could you please pull with a rebase so there aren’t intervening merge commits? If this passes, I’ll squash this one out for you.

@CodaFi
Copy link
Contributor

CodaFi commented Dec 11, 2019

@swift-ci please test

1 similar comment
@CodaFi
Copy link
Contributor

CodaFi commented Dec 11, 2019

@swift-ci please test

@LucianoPAlmeida
Copy link
Contributor Author

Thank you @CodaFi

In the future, though, could you please pull with a rebase so there aren’t intervening merge commits?

Sure :)

@CodaFi
Copy link
Contributor

CodaFi commented Dec 11, 2019

⛵️

@CodaFi CodaFi merged commit 392baef into swiftlang:master Dec 11, 2019
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.

3 participants