Thanks to visit codestin.com
Credit goes to github.com

`<mdspan>`: Various cleanups #3534

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

Merged

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

Mar 3, 2023

Contributor

JMazurkiewicz commented Mar 3, 2023

Renames:
- _Lhs -> _Left,
- _Rhs -> _Right,
- indexes -> indices (part of several identifiers),
- i -> _Ix (unreserved identifier),
- result -> _Result (also unreserved identifier).
Style:
- explicit(...) constexpr -> constexpr explicit(...) (looks like clang-format gives better result with the latter),
- Remove double empty lines,
- Always add empty line between functions,
- Reorder span and const array& constructors of std::extents to match the synopsis (see [mdspan.extents]),
- Add _Get_dynamic_indices function to _Mdspan_extent_type to avoid IIFE.

JMazurkiewicz added 2 commits

March 3, 2023 22:28


          Some cleanups

73dab82


          explicit(...) constexpr -> constexpr explicit(...)

1971d6b

JMazurkiewicz requested a review from a team as a code owner

March 3, 2023 21:48

StephanTLavavej added enhancement mdspan labels

StephanTLavavej approved these changes

View reviewed changes

StephanTLavavej merged commit 02b0ecb into microsoft:feature/mdspan2

JMazurkiewicz deleted the mdspan/style branch

March 3, 2023 23:19

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

enhancement mdspan