Re: [PATCH 00/22] syntax-check: Check misaligned stuff and fix them (batch II)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 2018-10-05 at 17:16, DanielP. Berrangé wrote:
>On Fri, Oct 05, 2018 at 11:11:28AM +0200, Andrea Bolognani wrote:
>> On Fri, 2018-10-05 at 09:31 +0100, Daniel P. Berrangé wrote:
>> > I've looked at clang-format before and it is pretty good. It does not
>> > have ability to quite match our code style, but I don't think that
>> > is a real show stopper. We'd be better off just adapting to one of
>> > the common code styles rather than inventing yet another app todo
>> > code indent analysis.
>>
>> I *strongly* support this idea :)
>
>The other benefit of clang-format is that it can actually fix the code
>automatically, as well as reporting on the problem. Our syntax check
>tool merely reports the problem leaving the developer tedious work to
>fix each problem.
>
>Regards,
>Daniel
>--
>|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
>|: https://libvirt.org -o- https://fstop138.berrange.com :|
>|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :| 

Don't know that before. Thanks!
The clang-format is really good and its style could be customized
according to libvirt.

Shi Lei

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux