majnemer added inline comments.

> majnemer wrote in PrintPreprocessedOutput.cpp:321-325
> Variables should start with uppercase characters.

Please uppercase all your other variables too.

> PrintPreprocessedOutput.cpp:344
> + */
> +bool tryGetTokenText(StringRef *text, const Token &tok) {
> +  if (tok.getKind() == clang::tok::identifier) {

Could this just return a StringRef? You could use an empty StringRef on failure.

https://reviews.llvm.org/D25153



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to