[PATCH] Change OwningPtr::take() to OwningPtr::release().

Manuel Klimek klimek at google.com
Mon Mar 10 22:50:39 PDT 2014


On Mon, Mar 10, 2014 at 10:58 PM, Ahmed Charles <acharles at outlook.com>wrote:

> ________________________________
> > Date: Sun, 9 Mar 2014 20:50:11 +0100
> > Subject: Re: [PATCH] Change OwningPtr::take() to OwningPtr::release().
> > From: klimek at google.com
> > To: reviews+D3000+public+98092d670b5699b2 at llvm-reviews.chandlerc.com
> > CC: dblaikie at gmail.com; chandlerc at gmail.com; acharles at outlook.com;
> > cfe-commits at cs.uiuc.edu
> >
> > clang-format should (if used with the right scripts) only re-format
> > statements around touched lines (it will mostly re-format full
> > statements though).
> >
> > What scripts did you use to clang-format the changes?
>
>
> I used git-clang-format from the tools/clang-format directory.
>


That should only format statements for which git thinks there's already a
diff - if you find this not to be true, please file bugs. Thanks!
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140311/8a26a319/attachment.html>


More information about the cfe-commits mailing list