<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Actually, here's one. The fake open function for the OSTC 2N. That could<br>
have been its own commit, added before the next commit that adds all the<br>
qtserialbluetooth stuff. That's a simple example, but maybe it illustrates<br>
what I mean.<br></blockquote><div><br></div><div>It is funny that in the beginning I did an isolated commit for that. In the end</div><div>I thought that it would be better to separate the patches in three groups:</div><div>- changes on the current UI</div><div>- implementation of the selection dialog</div><div>- implementation of our custom serial communication</div><div><br></div><div>I know that some of them contain too many lines of code but I thought </div><div>that it would be easier to group them by their main scope.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
I won't ask you to refactor the commits because I don't think there's<br>
anything wrong in what you did. Just something to look into for the<br>
future.<br>
<span class=""><br></span></blockquote><div>I will try to use a high degree <span style="font-size:12.8000001907349px">of granularity for the </span><span style="font-size:12.8000001907349px">commits in the future.</span></div><div><span style="font-size:12.8000001907349px">Thanks for your feedback!</span></div><div><span style="font-size:12.8000001907349px"><br></span></div><div><span style="font-size:12.8000001907349px">Claudiu</span></div></div></div></div>