Skip to content

<mdspan>: Various cleanups (mostly renames) - #3593

Merged
Stephan T. Lavavej (StephanTLavavej) merged 12 commits into
microsoft:feature/mdspan2from
JMazurkiewicz:mdspan/cleanups
Mar 27, 2023
Merged

<mdspan>: Various cleanups (mostly renames)#3593
Stephan T. Lavavej (StephanTLavavej) merged 12 commits into
microsoft:feature/mdspan2from
JMazurkiewicz:mdspan/cleanups

Conversation

@JMazurkiewicz

Copy link
Copy Markdown
Contributor
  • Fix feature test macro - it should be guarded with __cpp_lib_concepts,
  • Fix various typos,
  • Cleanup _EXPORT_STD - it should be on the same line as the class,
  • Update working draft revision to N4944 and add missing citations,
  • Prefer type aliases instead of _Ugly names (e.g. _Extents -> extents_type),
  • Replace _Mysomething names with _Something.

Comment thread tests/std/tests/P0009R18_mdspan_extents/test.cpp
Comment thread stl/inc/mdspan
@StephanTLavavej
Stephan T. Lavavej (StephanTLavavej) merged commit 3ea5312 into microsoft:feature/mdspan2 Mar 27, 2023
@StephanTLavavej

Copy link
Copy Markdown
Member

Thanks! Merged with a couple of comments for followup.

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

Labels

mdspan C++23 mdspan

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants