cplusplus / nbballot

Handling of NB comments in response to ballots
14 stars 4 forks source link

FR-016-027 24.7 [views] Reorganize subclauses #418

Closed wg21bot closed 1 year ago

wg21bot commented 1 year ago

The organisation of [views] is confusing.

Consider adding a global [multidimensional] section regrouping the description of all type pertaining to mdspan and future additions such as mdarray. As the wording is currently organised, there is no hierarchical relationship between mdspan, extent, layouts etc. Arguably, [span] could be moved under [ranges].

tkoeppe commented 1 year ago

Editorial meeting decision: accept, details to be worked out.

The new structure should be roughly:

tkoeppe commented 1 year ago

After some more thought, I wouldn't be opposed to a new subsection for span, too; [span.objectrep] should move up one level anyway and we would thus have three subclauses below that.

How about the names "Contiguous access" and "Multidimensional access" for the two new high-level subclauses?

@jwakely, @CaseyCarter: any thoughts on subclause names?

The proposed structure would thus be:

Views
+-- General
+-- Contiguous access
|   +-- Header <span>
|   +-- Class template `span`
|   +-- Views of object representations
+-- Multidimensional access
    +-- Overview
    +-- Header <mdspan>
    +-- ...
jensmaurer commented 1 year ago

Works for me.

tkoeppe commented 1 year ago

I pushed a proposed change to https://github.com/cplusplus/draft/pull/5942. We now have some level-6 subclauses, but I think that's still an improvement. Feedback welcome!

tkoeppe commented 1 year ago

Final disposition: Accepted.