Skip to content
This repository was archived by the owner on Jan 23, 2023. It is now read-only.

Remove unnecessary assert. #7889

Merged
merged 1 commit into from
Oct 29, 2016
Merged

Remove unnecessary assert. #7889

merged 1 commit into from
Oct 29, 2016

Conversation

@mazong1123
Copy link

@mazong1123 mazong1123 commented Oct 29, 2016

Remvoe the unneccessary assert that might cause a bug.

fix #6856

@mazong1123
Remove unnecessary assert.
f017a8c 
Remvoe the unneccessary assert that might cause a bug.

fix #6856
jkotas
jkotas approved these changes Oct 29, 2016
View reviewed changes
Copy link
Member

@jkotas jkotas left a comment

Choose a reason for hiding this comment

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

Thanks!

@jkotas jkotas merged commit d64c7d6 into dotnet:master Oct 29, 2016
@mazong1123 mazong1123 deleted the fix-6856 branch October 29, 2016 17:49
@karelz karelz modified the milestone: 2.0.0 Aug 28, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@jkotas jkotas jkotas approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

2.0.0

Development

Successfully merging this pull request may close these issues.

Possible assert in COMModule::GetStringConstant

4 participants

@mazong1123 @jkotas @karelz @dnfclas