Skip to content
This repository was archived by the owner on Oct 12, 2022. It is now read-only.

Issue 7020 - Exception thrown across DLL is not caught. #92

Merged
merged 1 commit into from
Dec 1, 2011
Merged

Issue 7020 - Exception thrown across DLL is not caught. #92

merged 1 commit into from
Dec 1, 2011

Conversation

shoo
Copy link
Contributor

@shoo shoo commented Nov 27, 2011

http://d.puremagic.com/issues/show_bug.cgi?id=7020
Added the comparison by the TypeInfo name.

@donc
Copy link

donc commented Nov 27, 2011

Does this also fix bug 1693? It sounds very similar.

@shoo
Copy link
Contributor Author

shoo commented Nov 27, 2011

Thank you for comment.
Issue 7020 is surely similar to Issue 1693.
The sample is old and does not work in the current environment, but is the try that is approximately equal to my sample as far as I check source. I'll mark Issue 7020 as duplication.
About Issue 1693, it will be fixed by this PullRequest, too.

@donc
Copy link

donc commented Nov 27, 2011

1693 might be different for D1.

@complexmath
Copy link
Member

I thought TypeInfo comparison by name had beed added ages ago. This one must have been missed.

complexmath added a commit that referenced this pull request Dec 1, 2011
@complexmath
Merge pull request #92 from shoo/fix7020
29f33bf 
Issue 7020 - Exception thrown across DLL is not caught.
@complexmath complexmath merged commit 29f33bf into dlang:master Dec 1, 2011
MartinNowak added a commit to MartinNowak/druntime that referenced this pull request Jan 30, 2012
@MartinNowak
Revert "Merge pull request dlang#92 from shoo/fix7020"
b7bdcdc 
This reverts commit 29f33bf, reversing
changes made to f11dd3e.

fixes Issue 7375
@MartinNowak MartinNowak mentioned this pull request Jan 30, 2012
Merged
andralex added a commit that referenced this pull request Jul 8, 2012
@andralex
Merge pull request #142 from dawgfoto/fix7375
047fa7c 
Revert "Merge pull request #92 from shoo/fix7020"
@John-Colvin John-Colvin mentioned this pull request Oct 11, 2019
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@shoo @donc @complexmath