<div dir="ltr">Thanks Dirk,<div><br></div><div>I have a at least one more code optimisation. How would I get the last master and merge my commits that</div><div>I have not formatted to patches. I was reading the documentation about merge vs. rebase and frankly I don't</div><div>get it at this point in time.</div></div><div class="gmail_extra"><br><div class="gmail_quote">2015-09-10 21:38 GMT+02:00 Dirk Hohndel <span dir="ltr"><<a href="mailto:dirk@hohndel.org" target="_blank">dirk@hohndel.org</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Guido.<br>
<br>
This was an excellent series of patches. Well done.<br>
<br>
I ended up rephrasing the commit messages a but, especially the commit<br>
titles, but that's something that I do a lot for other people as well.<br>
I may be a bit over zealous there, but I think the result makes it easier<br>
to track what's going on.<br>
<span class=""><br>
On Thu, Sep 10, 2015 at 07:54:29PM +0200, Guido Lerch wrote:<br>
</span><span class="">> I am done with my first round of Uemis patches.<br>
> I am new to this so please forgive me if I still don't get this 100%<br>
> correct.<br>
<br>
</span>They were very close. A couple of comments with your name and date, other<br>
than that nothing stood out.<br>
<span class=""><br>
> Dirk advised me to commit as much as possible but I still struggle to<br>
> make a commit if the commit would not end up in something that is<br>
> executable. Advise appreciated.<br>
<br>
</span>The granularity of the commits was perfect. I could look at them,<br>
understand what they do and apply them quite quickly.<br>
<br>
If at all possible it's preferable that every commit compiles. Otherwise<br>
git bisect becomes a major pain.<br>
<span class=""><br>
> The patches are ALL required to get a running Uemis download, again<br>
> happy to get feedback on how to structure patches better.<br>
<br>
</span>I think you did this very well.<br>
<span class=""><br>
> I've tested the code with 2 Uemis computers that are very different in<br>
> dive sites and length of dives as well as the additional information<br>
> provided by the uemis and found no problems, all dives loaded nicely.<br>
><br>
> There might still be memory leaks and I have to learn more about patching<br>
> and GIT in general,<br>
> so I would appreciate if you send me your comments and let me fix issues<br>
> myself as this helps<br>
> me in my learning process. I come from C# and I know that C and C++ guys<br>
> might laugh at us<br>
> as we have not a lot experience when it comes down to memory handling. I<br>
> want to learn that<br>
> too, so please give me a chance :-)<br>
<br>
</span>Sorry, I already fixed things up as it was all minor stuff - and that is<br>
my typical workflow. If it's small stuff, I just adjust as I apply the<br>
patches - just like I did for Anton earlier today when he forgot the hunk<br>
that changed a .h file.<br>
<br>
Sure, I could back out the changes and explain to you what you should<br>
change... but frankly, I'd rather see you do more development and work<br>
with git again for your next patch series.<br>
<br>
Keep them coming.<br>
<br>
Thanks<br>
<span class="HOEnZb"><font color="#888888"><br>
/D<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr">Best regards,<div>Guido</div></div></div>
</div>