r181997 - Add a more convenient interface to use clang-format.

Daniel Jasper djasper at google.com
Thu May 16 09:48:15 PDT 2013


Good point. Will look into it (for both functions).


On Thu, May 16, 2013 at 6:43 PM, Jordan Rose <jordan_rose at apple.com> wrote:

>
> On May 16, 2013, at 3:40 , Daniel Jasper <djasper at google.com> wrote:
>
> +/// \brief Reformats the given \p Ranges in \p Code.
> +///
> +/// Otherwise identical to the reformat() function consuming a \c Lexer.
> +tooling::Replacements reformat(const FormatStyle &Style, StringRef Code,
> +                               std::vector<tooling::Range> Ranges,
> +                               StringRef FileName = "<stdin>");
>
>
> Why copy all the ranges instead of using ArrayRef?
>
> Jordan
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130516/391118cc/attachment.html>


More information about the cfe-commits mailing list