[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