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 Issue 22674 - ImportC: compatible types declared in different tra… #14461

Merged
merged 1 commit into from Sep 21, 2022

Conversation

WalterBright
Copy link
Member

…nslation units are not treated equivalent in D

This fix seems too easy. Let's see how it works.

@WalterBright WalterBright added Easy Review ImportC Pertaining to ImportC support labels Sep 21, 2022
@dlang-bot
Copy link
Contributor

Thanks for your pull request, @WalterBright!

Bugzilla references

Auto-close Bugzilla Severity Description
22674 major ImportC: compatible types declared in different translation units are not treated equivalent in D.

Testing this PR locally

If you don't have a local development environment setup, you can use Digger to test this PR:

dub run digger -- build "master + dmd#14461"

…nslation units are not treated equivalent in D
Copy link
Member

@maxhaton maxhaton left a comment

Choose a reason for hiding this comment

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

lgtm, would like to see what Iain and Adam make of it

@WalterBright WalterBright merged commit 8d8f39f into dlang:master Sep 21, 2022
@WalterBright WalterBright deleted the fix22674 branch September 21, 2022 18:03
@adamdruppe
Copy link
Contributor

I wish you'd wait until we have a chance to actually test it before merging. This didn't actually fix the issue listed and if you could have let us test it, we would have told you that.

I came down with a nasty surprise flu and it put me behind schedule on code work but still, there shouldn't have been a rush to merge.

maxhaton pushed a commit to maxhaton/dmd that referenced this pull request Sep 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Fix Easy Review ImportC Pertaining to ImportC support
Projects
Status: Done
4 participants