<div dir="ltr"><div class="gmail_extra"><div><div class="gmail_signature"><div dir="ltr"><br></div></div></div>
<br><div class="gmail_quote">On Fri, Dec 1, 2017 at 11:18 PM, Dirk Hohndel <span dir="ltr"><<a href="mailto:dirk@hohndel.org" target="_blank">dirk@hohndel.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><div style="word-wrap:break-word">So much fun to wake up to such good news.<div>Thank you, Murillo!</div><div><br></div><div>I glanced through the diff and (like Miika) am not quite sure why you changed things to QString (if this is needed, could this be a separate commit?)</div></div></blockquote><div><br></div><div>Change to QStrings is not really necessary, just made it quicker for me to test what I really wanted to test.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><div style="word-wrap:break-word"><div>And of course the question is "why isn't the call to connectionListMode.addAddress(<wbr>)" not needed? </div></div></blockquote><div><br></div><div><div>It is needed, still there for newDC. This just removed from the list things we "know" (maybe not for a fact) are not DC. I haven't understood yet all the code involved, so I might be missing something.</div><div><br></div><div>Just to make BLE work this change is not really needed, sure.</div></div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><div style="word-wrap:break-word"><div>And likewise "why aren't we calling saveBtDeviceInfo(...) on the other platforms".</div><div><br></div></div></blockquote><div><br></div><div>I only found references to getBtDeviceInfo on a code specific for Mac and iOS (qt-ble.cpp:300). Not sure how this works on Mac since it is never populated.</div><div><br></div><div><br></div><div>It's not clear why for each device discovered, instead of just storing the DeviceInfo we instead use this info to create multiple other structures (like <span style="color:rgb(9,46,100)">serviceUuids, btDCs) and later try to reconstruct whatever is needed.</span></div>

<div><br></div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><div style="word-wrap:break-word"><div></div><div>I'd love to see a pull request - this makes it easy to get it tested on all of our platforms. </div></div></blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><div style="word-wrap:break-word"><div><br></div><div>I don't think you've submitted code to Subsurface before - it's fairly straight forward, CONTRIBUTING.md contains the basics and of course feel free to ask if you need help :-)</div><span class="gmail-HOEnZb"><font color="#888888"><div><br></div></font></span></div></blockquote><div><br></div><div><div>I'm working on a proper patch and will create a PR, with signed-off patches and all.</div></div><div><br></div><div><br></div><div><...></div><div><br></div><div>-- </div><div>Murillo</div></div></div></div>