<p dir="ltr"><br>
On Apr 2, 2016 18:14, "Dirk Hohndel" <<a href="mailto:dirk@hohndel.org">dirk@hohndel.org</a>> wrote:<br>
><br>
> So you add the dive pointer to the function signature, but you don't do<br>
> anything with it, creating an unused parameter. Why?</p>
<p dir="ltr">Next patch will do it.</p>
<p dir="ltr">The dive pointer is necessary to look up the cylinder, but this patch tries to avoid actually changing behavior.</p>
<p dir="ltr">So all of this patch is basically prep.</p>
<p dir="ltr">That said, it does change behavior in some ways: just by virtue of setting the cylinder number this patch triggers behavior changes, because having a cylinder index on the gas change will mean that it gets shown and saved.</p>
<p dir="ltr">Similarly, if the gas change cannot actually be matched up to any cylinder, it will be thrown away, again being technically a behavior change.</p>
<p dir="ltr">So it's not entirely just infrastructure with no behavior change at all, but this patch basically just does stricter checking and introduces behavior that you *could* get before by specifying the cylinder index.</p>
<p dir="ltr">The next patch that actually uses the cylinder gas mix information is a more fundamental behavior change, in the sense that it actually introduces behavior that you couldn't even force before.</p>
<p dir="ltr">Does that explain the patch?</p>
<p dir="ltr">   Linus</p>