[PATCH] D83296: [clang-format] Add a MacroExpander.

Marek Kurdej via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jul 22 14:18:57 PDT 2020


curdeius added inline comments.


================
Comment at: clang/lib/Format/MacroExpander.cpp:46
+
+  // Parse the token stream and return the corresonding Defintion object.
+  // Returns an empty definition object with a null-Name on error.
----------------
Nit: typo "corresponding Definition".


================
Comment at: clang/lib/Format/MacroExpander.cpp:110
+
+MacroExpander::~MacroExpander() {}
+
----------------
Why isn't it defaulted?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83296





More information about the cfe-commits mailing list