[libc++][spaceship][NFC] P1614R2: Status page - mark header synopsis sections as "Complete"

[H-G-Hristov]

Summary

Header synospis sections of P1614R2 are implemented by other items usually. For completeness, let's mark some of them as "Complete".

[H-G-Hristov]

@Mordante
There are a couple of items in the list: https://libcxx.llvm.org/Status/Spaceship.html which I am not sure about if there is anything to do about or just mark them as "Complete" or "Nothing to Do".

• container.requirements.general
• meta.trans.other

I think this patch completes my part of P1614R2. IMO There is nothing to do about ranges library. I'll double check and submit the patch to mark them as complete. Please let me know if there is anything else to do about P1614R2.

[Mordante]

In D152775#4416463, @H-G-Hristov wrote:

@Mordante
There are a couple of items in the list: https://libcxx.llvm.org/Status/Spaceship.html which I am not sure about if there is anything to do about or just mark them as "Complete" or "Nothing to Do".

Nothing to do, means there is no (hypothetical) change needed in the library. For example when I implement the TZDB parts of the library I will directly use the operators as defined in P1614, not what's in the TZDB paper. Had we implemented the TZDB change first we would have needed make changes, like we did for the calendar changes. So then it's not Nothing to do.

Some papers update the wording of the Standard to a new style, without changing the meaning for the wording. These changes are nothing to do. I hope that makes it clearer for you.

For these changes in the status table I probably wouldn't have listed the synopsis at all. This page is temporary and used for the libc++ developers to track the status. It will be removed in the future. So I don't feel to strong about Complete/Nothing to do here; I feel Complete is slightly better.

LGTM!

[H-G-Hristov]

In D152775#4421411, @Mordante wrote:

In D152775#4416463, @H-G-Hristov wrote:

@Mordante
There are a couple of items in the list: https://libcxx.llvm.org/Status/Spaceship.html which I am not sure about if there is anything to do about or just mark them as "Complete" or "Nothing to Do".

Nothing to do, means there is no (hypothetical) change needed in the library. For example when I implement the TZDB parts of the library I will directly use the operators as defined in P1614, not what's in the TZDB paper. Had we implemented the TZDB change first we would have needed make changes, like we did for the calendar changes. So then it's not Nothing to do.

Some papers update the wording of the Standard to a new style, without changing the meaning for the wording. These changes are nothing to do. I hope that makes it clearer for you.

For these changes in the status table I probably wouldn't have listed the synopsis at all. This page is temporary and used for the libc++ developers to track the status. It will be removed in the future. So I don't feel to strong about Complete/Nothing to do here; I feel Complete is slightly better.

LGTM!

Thank you for explaining.

libcxx/docs/Status/SpaceshipProjects.csv

 | `[optional.comp.with.t] <https://wg21.link/optional.comp.with.t>`_","| `optional <https://reviews.llvm.org/D146392>`_
 | `nullopt <https://reviews.llvm.org/D146392>`_",None,Hristo Hristov,|Complete|
 "| `[variant.syn] <https://wg21.link/variant.syn>`_
 | `[variant.relops] <https://wg21.link/variant.relops>`_
 | `[variant.monostate.relops] <https://wg21.link/variant.monostate.relops>`_","| `monostate <https://reviews.llvm.org/D131372>`_
 | `variant <https://reviews.llvm.org/D131372>`_",None,Kent Ross,|Complete|
 "| `[template.bitset] <https://wg21.link/template.bitset>`_
 | `[bitset.members] <https://wg21.link/bitset.members>`_","| remove ops `bitset <https://reviews.llvm.org/D152611>`_",None,Hristo Hristov,|Complete|
-| `[memory.syn] <https://wg21.link/memory.syn>`_,|,None,Unassigned,|Not Started|
+| `[memory.syn] <https://wg21.link/memory.syn>`_,|,None,Unassigned,|Complete|
 | `[allocator.globals] <https://wg21.link/allocator.globals>`_,| remove ops `allocator <https://reviews.llvm.org/D152612>`_,None,Hristo Hristov,|Complete|
 | `[unique.ptr.special] <https://wg21.link/unique.ptr.special>`_,| `unique_ptr <https://reviews.llvm.org/D130838>`_,[comparisons.three.way],Adrian Vogelsgesang,|Complete|
 | `[util.smartptr.shared.cmp] <https://wg21.link/util.smartptr.shared.cmp>`_,| `shared_ptr <https://reviews.llvm.org/D130852>`_,[comparisons.three.way],Adrian Vogelsgesang,|Complete|
 "| `[mem.res.syn] <https://wg21.link/mem.res.syn>`_
 | `[mem.res.eq] <https://wg21.link/mem.res.eq>`_
 | `[mem.poly.allocator.eq] <https://wg21.link/mem.poly.allocator.eq>`_","|
 | remove ops `memory_resource <https://reviews.llvm.org/D152629>`_
 | remove ops `polymorphic_allocator <https://reviews.llvm.org/D152629>`_",None,Hristo Hristov,|Complete|

 | remove ops `unordered_multimap <https://reviews.llvm.org/D152642>`_",None,Hristo Hristov,|Complete|
 | `[unordered.set.syn] <https://wg21.link/unordered.set.syn>`_,"| remove ops `unordered_set <https://reviews.llvm.org/D152643>`_
 | remove ops `unordered_multiset <https://reviews.llvm.org/D152643>`_",None,Hristo Hristov,|Complete|
 | `[queue.syn] <https://wg21.link/queue.syn>`_,| `queue <https://reviews.llvm.org/D146066>`_,None,Hristo Hristov,|Complete|
 | `[stack.syn] <https://wg21.link/stack.syn>`_,| `stack <https://reviews.llvm.org/D146094>`_,None,Hristo Hristov,|Complete|
 | `[queue.ops] <https://wg21.link/queue.ops>`_,| `queue <https://reviews.llvm.org/D146066>`_,None,Hristo Hristov,|Complete|
 | `[stack.ops] <https://wg21.link/stack.ops>`_,| `stack <https://reviews.llvm.org/D146094>`_,None,Hristo Hristov,|Complete|
 - `5.8 Clause 23: Iterators library <https://wg21.link/p1614r2#clause-23-iterators-library>`_,,,,
-| `[iterator.synopsis] <https://wg21.link/iterator.synopsis>`_,|,None,Unassigned,|Not Started|
+| `[iterator.synopsis] <https://wg21.link/iterator.synopsis>`_,|,None,Unassigned,|Complete|
 | `[reverse.iter.cmp] <https://wg21.link/reverse.iter.cmp>`_,| `reverse_iterator <https://reviews.llvm.org/D113695>`_,None,Mikhail Maltsev,|Complete|
 "| `[move.iterator] <https://wg21.link/move.iterator>`_
 | `[move.iter.op.comp] <https://wg21.link/move.iter.op.comp>`_",| `move_iterator <https://reviews.llvm.org/D117656>`_,None,Arthur O'Dwyer,|Complete|
 "| `[common.iterator] <https://wg21.link/common.iterator>`_
 | `[common.iter.cmp] <https://wg21.link/common.iter.cmp>`_",| `common_iterator <https://reviews.llvm.org/D103335>`_,None,Zoe Carver,|Complete|
 "| `[counted.iterator] <https://wg21.link/counted.iterator>`_
 | `[counted.iter.cmp] <https://wg21.link/counted.iter.cmp>`_",| `counted_iterator <https://reviews.llvm.org/D106205>`_,None,Zoe Carver,|Complete|
 | `[unreachable.sentinel] <https://wg21.link/unreachable.sentinel>`_,| `unreachable_sentinel_t <https://reviews.llvm.org/D107920>`_,None,Zoe Carver,|Complete|