<HTML><BODY><div id="composeWebView_editable_content" data-mailruapp-compose-id="composeWebView_editable_content" style="text-align: left;">Hi Dirk<div><br></div><div>Keep to see your cleanup, I know I am still struggling to forget about c# :-)</div><div><br></div><div>I think the code is good but have a small patch later today, I am still testing. This might avoid a few reads but not a lot.</div><div><br></div><div>Over the past two days I read another 10 times all my dives while trying to do some stupid things here and there such as canceling in the middle etc. found no issues but with the Uemis you never know.</div><div><br></div><div>Heading out to marsa alam tomorrow , if I find someone with an Uemis I'll read that to if I am allowed. This should give us some more data.</div><div><br></div><div>How did the tests with you Uemis go?<br><br>G. Lerch<br><br><br>Freitag, 18. September 2015 17:35 +0200 von Dirk Hohndel <dirk@hohndel.org>:<br> <div id="composeWebView_previouse_content" data-mailruapp-compose-id="composeWebView_previouse_content"><blockquote style="border-left: 1px solid #fc2c38; margin: 10px 10px 10px 5px; padding: 0 0 0 10px;"><div class="js-helper js-readmsg-msg">
<style type="text/css"></style>
<div>
<base target="_self" href="https://e-aj.my.com/">
<div id="style_14425905430000003501_BODY">On Fri, Sep 18, 2015 at 05:26:42PM +0200, Guido Lerch wrote:<br>
<div class="mail-quote-collapse">> All good now.<br>
> <br>
> I looked into avoiding to read all the dive-spots but this is not possible<br>
> unless the divespot was read in a current import<br>
> already. The reason on why we cannot compare already stored dives from the<br>
> xml is that we don't store the dive spot<br>
> id. I don't think we should implement this just for the uemis.<br>
> <br>
> What I am doing already is to avoid duplication of dive spots, and not<br>
> doing anything with dive-spots if the uemus dive-spot<br>
> id is -1. So Dirk, in your case not a single dive-spot is being read :-)<br>
> For me about 80.<br>
> <br>
> I keep on thinking however on how to make the import faster and allow<br>
> partial imports while I am on vacation :-)<br>
> <br>
> If someone is testing and finds an issue, please let me know.<br>
</div><br>
So if I read this correctly you think that the current code is "good to<br>
go" for 4.5?<br>
<br>
I may make a couple of tiny adjustments here and there (let's call them<br>
"cleanup"), but nothing significant.<br>
<br>
I'm planning to release Beta 2 today.<br>
<br>
/D<br>
</div>
<base target="_self" href="https://e-aj.my.com/">
</div>
</div></blockquote></div></div></div></BODY></HTML>