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.

Sign up for GitHub

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

Jump to bottom

`<mdspan>`: Cleanup `mdspan` class a little bit #3672

Merged

StephanTLavavej merged 2 commits into microsoft:feature/mdspan2 from JMazurkiewicz:mdspan/mdspan

Apr 26, 2023

Contributor

JMazurkiewicz commented Apr 26, 2023

Use direct-non-list initialization in constructors (as standard says),
Reorder members to match [mdspan.layout.stride],
Strengthen some member functions.


          Cleanup mdspan class a little bit

9ed58b2

JMazurkiewicz requested a review from a team as a code owner

April 26, 2023 13:02


          Fix another signed/unsigned mismatch 🙄

267d664

StephanTLavavej added the mdspan label

StephanTLavavej self-assigned this

StephanTLavavej approved these changes

View reviewed changes

StephanTLavavej merged commit 7899889 into microsoft:feature/mdspan2

JMazurkiewicz deleted the mdspan/mdspan branch

April 26, 2023 20:13

StephanTLavavej mentioned this pull request

P0009R18 <mdspan> #3972

Merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mdspan