[libcxx-commits] [PATCH] D132284: [libc++] Reduces the number of transitive includes.

[Kent Ross via Phabricator via libcxx-commits]

mumbleskates added inline comments.


================
Comment at: libcxx/docs/DesignDocs/HeaderRemovalPolicy.rst:34
+transitive includes at any other time, however new language versions will be
+used as a convenient way to perform bulk removals of
+
----------------
Sentence fragment terminates here


================
Comment at: libcxx/docs/ReleaseNotes.rst:62-64
+  in use. Note that in the future, libc++ reserves the right to remove
+  incidental transitive includes more aggressively, in particular regardless
+  of the language version in use.
----------------
Is "the future" here major releases? years? If this deeply affected me I would want to know a lot more about what kind of timeline I was guaranteed, given "regardless of the language version in use".


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D132284