[libcxx-commits] [PATCH] D116691: [libc++] Add Status page for P2321R2 (Zip)

[Arthur O'Dwyer via Phabricator via libcxx-commits]

Quuxplusone added a comment.

LGTM % comments... although this //would// be another place we could try using GitHub issues instead of .csv files, if we wanted to. (I'm neutral.)



================
Comment at: libcxx/docs/Status/Zip.rst:17-19
+is assigned to these sub-projects. This is imperitave to effective
+implementation so that work is not duplicated and implementors are not blocked
+by each other.
----------------
jloser wrote:
> `s/imperitave/imperative`
I'd just replace this whole sentence with `This avoids duplicating effort.` libc++ is hardly known for "effective implementation," ;) and I don't actually see how this has any effect on "implementors [getting] blocked by each other." In a sense, having this list (a synchronization mechanism) is what makes it //possible// for one person to get blocked by another; without it, you'd just duplicate the other person's work (which could lead to starvation of the slower worker, but never deadlock).


================
Comment at: libcxx/docs/Status/Zip.rst:23
+to the #libcxx channel in the LLVM discord. Please *do not* start working
+on any of the assigned items below.
+
----------------



Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D116691/new/

https://reviews.llvm.org/D116691