Re: include-what-you-use run on Gluster

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

 



Thanks for this initiative. This is very helpful. 

Also noticed that all of the coverity issues of 'INCLUDE_RECURSION' would also be solved by this list. I haven't analysed the whole list yet, but the first 1000s of lines revealed that this is very useful.

-Amar


On Tue, Dec 25, 2018 at 10:18 PM Yaniv Kaul <ykaul@xxxxxxxxxx> wrote:
Now that I've fixed the paths, the attached report is a more accurate representation.
Regretfully, it's not easy to apply as a patch.

On Tue, Dec 25, 2018 at 9:53 AM Yaniv Kaul <ykaul@xxxxxxxxxx> wrote:
I'm not sure I'm using it correctly and I'm having trouble compiling it from source on my F29 machine, but include-what-you-use (https://include-what-you-use.org ) allows you to easily analyze your include statements and ensure accuracy. I believe it has found some issues in our code - I certainly could remove some unneeded includes - but I'm unsure if I need to follow all of its advises to make things work, otherwise something will surely break.

Attached please find the full analysis - hope it's useful for someone. I believe once we figure out how to use it properly, it can be a good tool - will probably reduce compile time in a negligible way too.

Y.

_______________________________________________
Gluster-devel mailing list
Gluster-devel@xxxxxxxxxxx
https://lists.gluster.org/mailman/listinfo/gluster-devel


--
Amar Tumballi (amarts)
_______________________________________________
Gluster-devel mailing list
Gluster-devel@xxxxxxxxxxx
https://lists.gluster.org/mailman/listinfo/gluster-devel

[Index of Archives]     [Gluster Users]     [Ceph Users]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux