[PATCH] Add a Mozilla coding style to clang-format

[Manuel Klimek]

On Mon, May 6, 2013 at 4:06 PM, Alexander Kornienko <alexfh at google.com>wrote:

> No, it doesn't. The only thing that bothers me a bit is that we don't have
> any tests for this style. There's plenty of tests for LLVM style in form of
> unit tests, but we still need some kind of file-based integration tests for
> different style guides.
>

As styles are purely declarative, I'm not that worried about not having
tests (I agree they might be nice to have though).


> Anyway, I'm going to commit your patch with getMozillaStyle being based
> upon getLLVMStyle: http://llvm-reviews.chandlerc.com/D750. If you have
> any comments, you can share them on Phabricator.
>
>
> On Mon, May 6, 2013 at 3:14 PM, Ehsan Akhgari <ehsan.akhgari at gmail.com>wrote:
>
>> Does that mean that we can't take this patch in the mean time?
>>
>> --
>> Ehsan
>> <http://ehsanakhgari.org/>
>>
>>
>> On Mon, May 6, 2013 at 9:07 AM, Alexander Kornienko <alexfh at google.com>wrote:
>>
>>> I'm starting to work on configuration files for clang-format, which are
>>> going to be a more flexible solution for per-project coding styles.
>>>
>>>
>>>  On Sun, May 5, 2013 at 9:28 PM, Ehsan Akhgari <ehsan.akhgari at gmail.com>wrote:
>>>
>>>>  Can someone please review and commit this patch?  Thanks!
>>>>
>>>> --
>>>> Ehsan
>>>> <http://ehsanakhgari.org/>
>>>>
>>>> _______________________________________________
>>>> cfe-commits mailing list
>>>> cfe-commits at cs.uiuc.edu
>>>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>>>>
>>>>
>>>
>>>
>>> --
>>> Alexander Kornienko | Software Engineer | alexfh at google.com | +49 151
>>> 221 77 957
>>> Google Germany GmbH | Dienerstr. 12 | 80331 München
>>>
>>
>>
>
>
> --
> Alexander Kornienko | Software Engineer | alexfh at google.com | +49 151 221
> 77 957
> Google Germany GmbH | Dienerstr. 12 | 80331 München
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130506/5d41b861/attachment.html>