Skip to content

Conversation

@JMazurkiewicz
Copy link
Contributor

@JMazurkiewicz JMazurkiewicz commented Mar 17, 2023

Short motivation:

  • Separate tests are easier to read and maintain,
  • We avoid files with 1000 lines of code.

That said, this PR adds:

P0009R18_mdspan_extents/test.cpp tests have two FIXME comment, as there might be a bug in extents implementation

@JMazurkiewicz JMazurkiewicz requested a review from a team as a code owner March 17, 2023 16:49
@StephanTLavavej StephanTLavavej added the mdspan C++23 mdspan label Mar 17, 2023
@StephanTLavavej StephanTLavavej self-assigned this Mar 17, 2023
@StephanTLavavej StephanTLavavej merged commit 404605b into microsoft:feature/mdspan2 Mar 17, 2023
@StephanTLavavej
Copy link
Member

Thanks for extending this test coverage! 😹

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