[libcxx-commits] [PATCH] D133127: [libc++][test] Improves transitive includes.

[Mark de Wever via Phabricator via libcxx-commits]

Mordante marked 4 inline comments as done.
Mordante added inline comments.


================
Comment at: libcxx/test/libcxx/transitive_includes_to_csv.py:23
+
+def parse_line(line: str) -> header:
+    match = re.match(r"(\.+) (.+)", line)
----------------
ldionne wrote:
> Python has a `csv` module you could use instead.
I had a look at it and it doesn't seem to be very useful in this case. Here we only write CSV to the output.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D133127