<div dir="ltr"><div class="gmail_extra"><div><div class="m_-2104764493874849494gmail_signature"><div dir="ltr"><br></div></div></div>
<br><div class="gmail_quote">On Sat, Dec 2, 2017 at 12:01 AM, 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"><br><div><span class="m_-2104764493874849494gmail-"><blockquote type="cite"><div>On Dec 1, 2017, at 7:50 AM, Murillo Bernardes <<a href="mailto:mfbernardes@gmail.com" target="_blank">mfbernardes@gmail.com</a>> wrote:</div><div><div dir="ltr"><div class="gmail_extra"><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></div></div></blockquote><div><br></div></span>OK</div><div><span class="m_-2104764493874849494gmail-"> <br><blockquote type="cite"><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-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></div></div></blockquote><div><br></div></span><div>I'm not sure I understand your answer. But then I should stare at the code in context some more.</div><span class="m_-2104764493874849494gmail-"><br><blockquote type="cite"><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-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></blockquote><div><br></div></div></div></div></blockquote></span></div></div></blockquote><div><br></div><div>On desktop we are calling saveBtDeviceInfo(...) when the Save button is clicked.</div><div><br></div><div>Now I understand why even though the last device is showed it doesn't work unless I scan and save again.</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><span class="m_-2104764493874849494gmail-"><blockquote type="cite"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div></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></div></div></blockquote><div><br></div></span><div>I think it is fair to say that this code went through a lot of iterations between the initial ideas and what ended up working across all of the platforms. It may be a good idea to look into cleaning things up and simplifying the data structures, once we have it all working.</div><span class="m_-2104764493874849494gmail-"><br></span></div></div></blockquote><div><br></div><div>For now I sent a PR just saving all DeviceInfo on iOS.</div><div><br></div><div>I'm planning to modify ConnectionListModel in order to store additional data, in the same way it's done for desktop.</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><span class="m_-2104764493874849494gmail-"><blockquote type="cite"><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-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="m_-2104764493874849494gmail-m_5409934136174640585gmail-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></div></div></blockquote><br></span></div><div>Thank you. Much appreciated</div><span class="m_-2104764493874849494gmail-HOEnZb"><font color="#888888"><div><br></div><div>/D</div><br></font></span></div></blockquote></div><br></div></div>