r210013 - clang-format: No space between ")" and braced init list.

Daniel Jasper djasper at google.com
Tue Jun 10 10:47:58 PDT 2014


Yes it did.. Remove the trailing semicolon and it will work fine again.. I
might eventually fix this to work in spite of the semicolon, but that is
VERY low priority..
On Jun 10, 2014 6:17 PM, "Adam Strzelecki" <ono at java.pl> wrote:

> Daniel,
>
> I believe this commit broke formatting removing space before function body
> and ")", running latest clang-format on my code introduced following
> unexpected changes:
>
>    Kernel(Detector<F> detector, ErrorsModel<F> errors_model)
> -      : detector_(detector), errors_model_(errors_model) {};
> +      : detector_(detector), errors_model_(errors_model){};
>
>  template <int N, typename F> class Vector {
>   public:
> -  Vector() {};
> +  Vector(){};
>
>  template <int N, typename F = double>
> -Point<N, F> operator+(const Vector<N, F>& v, const Point<N, F>& p) {};
> +Point<N, F> operator+(const Vector<N, F>& v, const Point<N, F>& p){};
>
>
> Can you please have a look?
>
> Regards,
> --
> Adam
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140610/cc3cc36d/attachment.html>


More information about the cfe-commits mailing list