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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits