Questions about code reviews and static analysis tools for TU Delft research

[Stephan Bergmann]

On 02/24/2015 11:53 AM, Radjino Bholanath wrote:
> 1. Do all developers (contributors and core developers) have to submit a code review for every change? I’m asking because many projects only review changes made by contributors.
> 2. Which code review tools are used?

<http://gerrit.libreoffice.org>, mandatory for people w/o commit access, 
encouraged for the others.

> 3. Are static analyzers used? If they are used:
>    a. Is passing the checks of the static analyzers necessary for a change to be accepted?

No, static analyzers are run asynchronously from the commit process.

>    b. Which static analyzers are used?

* <https://scan.coverity.com/projects/211>

* <https://wiki.documentfoundation.org/Development/Cppcheck>

* A set of project-specific Clang plugins, 
<https://wiki.documentfoundation.org/Development/Clang_plugins>