[cfe-dev] [Clang 4.0 Docs] AlignConsecutiveDeclarations Improvement Proposal

[David Wiberg via cfe-dev]

Hi,

If you haven't already, please also see
http://llvm.org/docs/DeveloperPolicy.html which contains some more
practical information.

Best regards
David

2016-12-23 4:45 GMT+01:00 Nikola Smiljanic via cfe-dev <cfe-dev at lists.llvm.org>:
> Absolutely, fix
> http://llvm.org/svn/llvm-project/cfe/trunk/docs/ClangFormatStyleOptions.rst
> and upload the patch to Phabricator and cc cfe-commits mailing list.
>
> On Thu, Dec 22, 2016 at 9:06 PM, FIXED-TERM Kammel Fabian (ETAS-PSC/EPE) via
> cfe-dev <cfe-dev at lists.llvm.org> wrote:
>>
>> Hey,
>>
>>
>>
>> The other day I asked a question about clang-format on stackoverflow [1].
>> And another user called attention to the fact that, in the section about
>> AlignConsecutiveDeclarations, the types of the initializers and variables do
>> not match. It reads:
>>
>>
>>
>> int         aaaa = 12;
>>
>> float       b = 23;
>>
>> std::string ccc = 23;
>>
>>
>>
>> And maybe should be changed to something like:
>>
>>
>>
>> int         aaaa = 12;
>>
>> float       b = 2.4f;
>>
>> std::string ccc = “some string”;
>>
>>
>>
>> Is the mailing list the correct place for suggestions like that? Or is
>> there another place to discuss improvements of the documentation?
>>
>>
>>
>> I try to integrate clang-format and clang-tidy in our development process
>> and might stumble upon more ambiguities and possible improvements.
>>
>>
>>
>> Also I noticed that the documentation (especially for clang-tidy options)
>> is sometimes quite sparse. Is there a way for me to contribute to the
>> documentation, since I need to figure this stuff out either way.
>>
>>
>>
>> Best regards,
>>
>> Fabian Kammel
>>
>>
>>
>> [1]
>> http://stackoverflow.com/questions/41264554/multiple-successive-declarations-definitions-on-same-indentation-level/41264995#41264995
>>
>>
>> _______________________________________________
>> cfe-dev mailing list
>> cfe-dev at lists.llvm.org
>> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev
>>
>
>
> _______________________________________________
> cfe-dev mailing list
> cfe-dev at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev
>