<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Feb 24, 2015 at 2:52 AM, Radjino Bholanath <span dir="ltr"><<a href="mailto:R.M.R.Bholanath@student.tudelft.nl" target="_blank">R.M.R.Bholanath@student.tudelft.nl</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi everyone,<br>
<br>
I'm doing research on code reviews and static analysis tools at the SERG group (<a href="http://swerl.tudelft.nl/bin/view/Main/WebHome" target="_blank">http://swerl.tudelft.nl/bin/view/Main/WebHome</a>) of the Delft University of Technology. Currently, we want to give an overview of the usage of code review and static analysis tools in open source projects. Therefore, I would be very happy to know a little bit more about how code reviews are used in Clang and if (and maybe how) static analysis tools are used. I have a couple of questions for anyone willing to answer:<br>
<br>
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.<br></blockquote><div><br>Not exactly (& not sure what definition of 'contributor' you are using). Pre-commit review is used by anyone who doesn't have commit rights and used at best judgment by those who do (if you're committing to an unfamiliar area, or a large/contentious patch, you might send it for review even though you have commit access). Post-commit review happens on everything (we just reply to auto-generated commit emails with any extra feedback we have) though it's not enforced in any way. (nothing checks that every commit is actually reviewed)<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
2. Which code review tools are used?<br></blockquote><div><br>Email and sometimes Phabricator.<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
3. Are static analyzers used? If they are used:<br></blockquote><div><br>Not really<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
  a. Is passing the checks of the static analyzers necessary for a change to be accepted?<br></blockquote><div><br>No<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
  b. Which static analyzers are used?<br></blockquote><div><br>I don't think we have any bots running any SA systems, but I guess some people run them manually from time to time and cleanup things.<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Thanks,<br>
<br>
Radjino<br>
<br>
_______________________________________________<br>
cfe-dev mailing list<br>
<a href="mailto:cfe-dev@cs.uiuc.edu">cfe-dev@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev</a><br>
</blockquote></div><br></div></div>