[PATCH] D13166: Create modernize-make-unique check.

Manuel Klimek via cfe-commits cfe-commits at lists.llvm.org
Mon Sep 28 02:49:18 PDT 2015


klimek added a comment.

In http://reviews.llvm.org/D13166#254520, @angelgarcia wrote:

> Two tests in which 'getTokenLength' returns 0.


Thanks for the tests - question is: I would have expected us to use something like Lexer::getSourceText, which should give us the full range in the first test (do we not want to replace it?) and tell us that we don't have the type in one consecutive range in the second case.


http://reviews.llvm.org/D13166





More information about the cfe-commits mailing list