[PATCH] Refactoring, update MacroInfo interface so a for range can be used to iterate through the tokens
Alexander Kornienko
alexfh at google.com
Fri May 8 03:19:23 PDT 2015
Sorry for the delay. I was on vacation.
Please address the comment and this should be fine to submit unless Richard has any comments.
================
Comment at: include/clang/Lex/MacroInfo.h:247
@@ -246,2 +246,3 @@
bool tokens_empty() const { return ReplacementTokens.empty(); }
+ const ArrayRef<Token> tokens() const { return ReplacementTokens; }
----------------
ArrayRef is pretty const on its own (in the sense that it doesn't allow modification of the elements in the range), so an extra const before ArrayRef<...> is not needed here.
http://reviews.llvm.org/D9079
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the cfe-commits
mailing list