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

On Thu, May 16, 2013 at 6:43 PM, Jordan Rose <[email protected]> wrote:

>
> On May 16, 2013, at 3:40 , Daniel Jasper <[email protected]> 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
>
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to