[PATCH] clang-format.el: customization-support, MELPA-compatibility

Manuel Klimek klimek at google.com
Thu Jan 8 07:32:22 PST 2015


+cfe-commits (please do not cut that out during code review, the list is
the source of truth for code reviews)

Landed as r225447. Note that I'm planning to adapt it soon to not replace
the whole buffer, but just apply the diffs (I'll cc' you on the patch for
review, unless you object :)

Cheers & thx!
/Manuel

On Wed Jan 07 2015 at 6:06:08 PM Johann Klähn <kljohann at gmail.com> wrote:

> On Wed, Jan 7, 2015 at 5:14 PM, Manuel Klimek <klimek at google.com> wrote:
> > +If called interactively uses the region or the current buffer if there
> > +is no active region.  If no style is given uses `clang-format-style'."
> >
> > Doing full-buffer if nothing is selected is a usability problem if one
> works
> > in an existing codebase; if no region is selected, it should just use the
> > cursor position (which will get the current statement reflowed).
> > The idea is that the user can do clang-format-buffer if they insist.
> Excellent point.
>
> > +(put 'clang-format-executable 'risky-local-variable t)
> >
> > Any reason not to use :risky in the defcustom?
> Oh, I thought I already did so. I must have accidently reverted that
> change.
>
> Thanks for your feedback, find attached an updated patch.
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150108/50c12cac/attachment.html>


More information about the cfe-commits mailing list