[mdspan.submdspan.overview] para 2.5, OK, more legible now.
Requested changes
In Section 3.1, please highlight additions in green (for changes in [contents]).
For hidden friends, declarations should appear in the class synopses as well (they already do; that declaration is correct). We don't want that mapping:: scope because it's not valid C++. Otherwise, the coalesced wording is fine. We could even just remove the mapping:: and put each declaration as a friend in its relevant mapping. We can still describe them here. Wait until later; Jonathan Wakely will e-mail us with a suggestion (the template parameters as displayed aren't correct). There may actually be an outstanding LWG issue about wording for hidden friends.
@crtrott @dalg24
Most recent changes reviewed
Changes that were OK
Requested changes
mapping::
scope because it's not valid C++. Otherwise, the coalesced wording is fine. We could even just remove themapping::
and put each declaration as a friend in its relevant mapping. We can still describe them here. Wait until later; Jonathan Wakely will e-mail us with a suggestion (the template parameters as displayed aren't correct). There may actually be an outstanding LWG issue about wording for hidden friends.