Skip to content

fix parameter name in general example description #4179

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
merged 1 commit into from
Mar 26, 2018

Conversation

robstoll
Copy link
Contributor

I suppose it should be x1 and xN instead of e1 and eN but maybe I misunderstand something because I don't see why only an upper approximation is possible.

I suppose it should be `x1` and `xN` instead of `e1` and `eN` but maybe I misunderstand something because I don't see why only an upper approximation is possible.
Copy link
Member

@dottybot dottybot left a comment

Choose a reason for hiding this comment

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

Hello, and thank you for opening this PR! 🎉

All contributors have signed the CLA, thank you! ❤️

Have an awesome day! ☀️

Copy link
Contributor

@Blaisorblade Blaisorblade left a comment

Choose a reason for hiding this comment

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

Agree with the fix — e1 and eN aren't even in scope.

@Blaisorblade
Copy link
Contributor

I don't see why only an upper approximation is possible.

On the run but that's a good question. Better: that has to be an upper approximation, but how do you handle contravariant occurrences of xi.T and xi.type in the result type? For xi.T you can take the lower bound of the type member, but what about xi.type? And what about invariant occurrences?

@Blaisorblade Blaisorblade merged commit ca6e2cc into scala:master Mar 26, 2018
@Blaisorblade
Copy link
Contributor

Merging this, can't investigate on the rest this week.

@robstoll robstoll deleted the patch-4 branch February 27, 2020 10:32
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