<div dir="ltr">On Tue, Feb 24, 2015 at 5:53 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><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);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 VLC 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></div><div>No, but developers with commit access will often send patches to the mailing list for discussion as needed (i.e. for clarifications or for major changes). Developers without commit access will have to send patches to the mailing list for review and to be merged by someone with access.<br></div><div>Details here: <a href="https://wiki.videolan.org/Sending_Patches_VLC/">https://wiki.videolan.org/Sending_Patches_VLC/</a><br></div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
2. Which code review tools are used?<br></blockquote><div><br></div><div>The mailing list and patchwork (see <a href="http://patches.videolan.org">patches.videolan.org</a>).<br> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
3. Are static analyzers used? If they are used:<br>
  a. Is passing the checks of the static analyzers necessary for a change to be accepted?<br></blockquote><div><br></div><div>No. Coverity scan is only used on code after it's been merged, afaik.<br> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
  b. Which static analyzers are used?<br></blockquote><div><br></div><div>Coverity scan, also I (and probably others) use scan-build from time to time.<br><br></div><div>Best,<br></div><div>Tristan<br></div></div></div></div>