<p dir="ltr">On 27 Jul 2016 1:19 a.m., "Hal Finkel" <<a href="mailto:hfinkel@anl.gov">hfinkel@anl.gov</a>> wrote:<br>
> Like what? Minor things are generally things that are easy to ask to be changed in a code review.</p>
<p dir="ltr">Minor problems like not using range based loops, too much use of std string, variable naming. Each own could be a simpler task, but rebasing (and validating) a large patch set for every review can be daunting. </p>
<p dir="ltr">I particularly don't feel strongly about caps on first letter of variable names, for instance, so I would be fine if a big renaming happening after landing, if the developer is already making other big and more important changes to the code base.</p>
<p dir="ltr">Cheers, <br>
Renato </p>