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

fix ICE Bug7212 #604

Merged
merged 1 commit into from
Jan 5, 2012
Merged

fix ICE Bug7212 #604

merged 1 commit into from
Jan 5, 2012

Conversation

MartinNowak
Copy link
Member

  • Only check covariant conversion to Type::nextOf
    for delegates and pointers.
    • With t->ty == Tfunction this would have
      checked conversion to the return type.

 - Only check covariant conversion to Type::nextOf
   for delegates and pointers.

 - With t->ty == Tfunction this would have
   checked conversion to the return type.
@MartinNowak
Copy link
Member Author

@9rnsr
Copy link
Contributor

9rnsr commented Jan 4, 2012

Ouch! Thanks for your patch.

@MartinNowak
Copy link
Member Author

Thanks too for fixing my recently introduced regression.

We should probably agree on a convention where/when
to check for null pointers. Lots of dmd functions do work when
being called with null, covariant apparently doesn't.

WalterBright added a commit that referenced this pull request Jan 5, 2012
@WalterBright WalterBright merged commit 34bdd68 into dlang:master Jan 5, 2012
braddr pushed a commit to braddr/dmd that referenced this pull request Oct 22, 2012
Issue 7898 - [CTFE] std.algorithm:copy fails when used with two arrays
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