<div dir="ltr">Hi,<div><br></div><div>we should decide what to do with whitespaces before the 4.6. release. The reason for the timing is that, it would make merging patches between 4.6 and master easier if we fix all whitespaces the same for the 4.6 and master branch. Erik would prefer not to do it after 4.6 so that we don&#39;t change most lines between 4.6 and 4.6.1. Also we probably don&#39;t have too many development branches at the moment, causing less problems if we do it now.</div>

<div><br></div><div>The discussion is at <a href="http://redmine.gromacs.org/issues/845">http://redmine.gromacs.org/issues/845</a> and a suggested solution is at: <a href="https://gerrit.gromacs.org/#/c/1680/">https://gerrit.gromacs.org/#/c/1680/</a></div>

<div><br></div><div>The patch is for master so that one can see also new (C++) files in master, but as written above, this change would also be applied to 4.6.</div><div><br></div><div>Open questions:</div><div>- Do we want to make the change now before the 4.6. release?</div>

<div>- Do we want to use an automatic tool to fix whitespaces at all?</div><div>- Do we want to use uncrustify or some other tool?</div><div>- Do we want to use the suggested uncrustify configuration?</div><div style>- Should all files or only the badly formated files be part of the intial fixup?</div>

<div>- Should it be optional or required to run the tool after the initial fixup?<br clear="all"><div style>- Should the tool be run automatically by Jenkins? </div><div style><br></div><div style>Roland</div>-- <br>ORNL/UT Center for Molecular Biophysics <a href="http://cmb.ornl.gov">cmb.ornl.gov</a><br>

865-241-1537, ORNL PO BOX 2008 MS6309
</div></div>