[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