<div dir="ltr">Adding formatting and other corrections which are unrelated to the functionality of the patch increases the noise in the patch when deltas are viewed in patch review and revision control tools. A few lines of formatting change are probably acceptable as long as they are a lesser portion of the patch, however if they are the majority it makes patch review and future debugging much more difficult and increases the likelihood that critical changes may be missed or obscured. Therefore functionality and formatting should be separate patches.<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, May 28, 2015 at 12:28 PM, Melanie <span dir="ltr"><<a href="mailto:melanie@t-data.com" target="_blank">melanie@t-data.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">That could happen if hardtabs are used. OpenSim's standard is<br>
4-space soft tabs. If indentation was removed or hardtabs<br>
introduced, that would need to be reversed.<br>
<span class="HOEnZb"><font color="#888888"><br>
- Melanie<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
On 28/05/2015 21:21, Cinder Roxley wrote:<br>
> On May 28, 2015 at 12:54:11 PM, Maxwell, Douglas CIV USARMY ARL (US) (<a href="mailto:douglas.maxwell3.civ@mail.mil">douglas.maxwell3.civ@mail.mil</a>) wrote:<br>
> Classification: UNCLASSIFIED<br>
> Caveats: NONE<br>
><br>
> As a rule, we are adhering to the coding standards of the target project.<br>
> In this case our guide is here:<br>
> <a href="http://opensimulator.org/wiki/Coding_standards" target="_blank">http://opensimulator.org/wiki/Coding_standards</a><br>
><br>
> Our team has been correcting any improperly formatted code as we come across<br>
> it. We are finding a lot of mismatched formatting and we believed we were<br>
> doing your community a service by performing these corrections as we spotted<br>
> them.<br>
><br>
> If these standards are no longer a part of your standard operating<br>
> procedures, please update your coding standards documentation and we will<br>
> adjust accordingly.<br>
><br>
> Thank you for your comment.<br>
><br>
> v/r -doug<br>
><br>
><br>
> Hi,<br>
><br>
> I assume he was referring to the areas that were changed which do *not* conform to either the opensim coding standing or any coding standard I’ve ever seen where indentation was removed making the code much harder to grok such as:<br>
><br>
> <a href="https://github.com/opensim/opensim/blob/master/OpenSim/Region/Framework/Scenes/Scene.cs#L3820" target="_blank">https://github.com/opensim/opensim/blob/master/OpenSim/Region/Framework/Scenes/Scene.cs#L3820</a><br>
> <a href="https://github.com/opensim/opensim/blob/master/OpenSim/Region/Framework/Scenes/Scene.cs#L4469" target="_blank">https://github.com/opensim/opensim/blob/master/OpenSim/Region/Framework/Scenes/Scene.cs#L4469</a><br>
> <a href="https://github.com/opensim/opensim/blob/master/OpenSim/Region/Framework/Scenes/Scene.cs#L4472" target="_blank">https://github.com/opensim/opensim/blob/master/OpenSim/Region/Framework/Scenes/Scene.cs#L4472</a><br>
><br>
> and various other places scattered about throughout Scene.cs between revisions c3138f9 and 625452b. It seems a lot of unbracketed single statement if blocks, even multi-lined ones, had their indentation stripped off. This is not a normal convention is any language I know of.<br>
><br>
> --<br>
> Cinder Roxley<br>
> Sent with Airmail<br>
><br>
><br>
><br>
</div></div><div class="HOEnZb"><div class="h5">> _______________________________________________<br>
> Opensim-dev mailing list<br>
> <a href="mailto:Opensim-dev@opensimulator.org">Opensim-dev@opensimulator.org</a><br>
> <a href="http://opensimulator.org/cgi-bin/mailman/listinfo/opensim-dev" target="_blank">http://opensimulator.org/cgi-bin/mailman/listinfo/opensim-dev</a><br>
_______________________________________________<br>
Opensim-dev mailing list<br>
<a href="mailto:Opensim-dev@opensimulator.org">Opensim-dev@opensimulator.org</a><br>
<a href="http://opensimulator.org/cgi-bin/mailman/listinfo/opensim-dev" target="_blank">http://opensimulator.org/cgi-bin/mailman/listinfo/opensim-dev</a><br>
</div></div></blockquote></div><br></div>