<p dir="ltr">I can see why it is important to have a well formatted code. Will pay more attention to it</p>
<p dir="ltr">Did you see the other one, about the trip headers? </p>
<br><div class="gmail_quote"><div dir="ltr">Il giorno Lun 5 Ott 2015 17:41 Dirk Hohndel <<a href="mailto:dirk@hohndel.org">dirk@hohndel.org</a>> ha scritto:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Mon, Oct 05, 2015 at 04:34:13PM +0100, Dirk Hohndel wrote:<br>
> On Mon, Oct 05, 2015 at 05:19:35PM +0200, Giorgio Marzano wrote:<br>
> > Dirk,<br>
> ><br>
> > this patch should fix the wrong values in the max depth field for splitted<br>
> > dives.<br>
><br>
> It does - I already have a more complete patch in my tree but haven't had<br>
> a chance to push that out, yet. Sorry about that.<br>
><br>
> I'll take your patch and refactor mine on top of it :-)<br>
><br>
> Patch looks good - maybe except for the two consecutive empty lines.<br>
> The commit message - I prefer the commit title to be in present tense (but<br>
> don't always catch it when people write it in past tense).<br>
<br>
Oh, and the missing space on both sides of '=' - I'll push the commit<br>
soon, please look at what ended up in master and compare to what you sent<br>
<br>
I know these are tiny nit-picks, but consistency is really important to<br>
make the code easy to read and intuitive to understand.<br>
<br>
/D<br>
</blockquote></div>