Skip to content

<complex>: prevent invalid conversions - #1294

Merged
Stephan T. Lavavej (StephanTLavavej) merged 5 commits into
microsoft:masterfrom
Arzaghi:Fix_Issue1256
Sep 23, 2020
Merged

<complex>: prevent invalid conversions#1294
Stephan T. Lavavej (StephanTLavavej) merged 5 commits into
microsoft:masterfrom
Arzaghi:Fix_Issue1256

Conversation

@Arzaghi

@Arzaghi Hamid Reza Arzaghi (Arzaghi) commented Sep 15, 2020

Copy link
Copy Markdown
Contributor

Fixes #1256

@Arzaghi Hamid Reza Arzaghi (Arzaghi) changed the title remove some of the constructors just a draft Sep 15, 2020
@Arzaghi Hamid Reza Arzaghi (Arzaghi) changed the title just a draft <complex>: prevent invalid conversions Sep 15, 2020
@Arzaghi
Hamid Reza Arzaghi (Arzaghi) marked this pull request as ready for review September 16, 2020 00:18
@StephanTLavavej Stephan T. Lavavej (StephanTLavavej) added the bug Something isn't working label Sep 16, 2020
Comment thread tests/libcxx/expected_results.txt
Comment thread stl/inc/complex

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Perfect, thanks! 😺

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nice: an all-red PR. 🔥🔥🔥 Burn it all down! 🔥🔥🔥

@StephanTLavavej
Stephan T. Lavavej (StephanTLavavej) merged commit 5847f82 into microsoft:master Sep 23, 2020
@StephanTLavavej

Copy link
Copy Markdown
Member

Thanks for fixing this bug, slightly improving compiler throughput, and improving our libcxx pass rate! 😸

@Arzaghi
Hamid Reza Arzaghi (Arzaghi) deleted the Fix_Issue1256 branch September 23, 2020 02:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

<complex>: invalid conversions allowed

3 participants