<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">2015-09-17 15:31 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"><span class="">On Thu, Sep 17, 2015 at 03:24:39PM +0200, Guido Lerch wrote:<br>
> 2015-09-17 14:20 GMT+02:00 Dirk Hohndel <<a href="mailto:dirk@hohndel.org">dirk@hohndel.org</a>>:<br>
><br>
> > On Thu, Sep 17, 2015 at 12:13:06PM +0200, Guido Lerch wrote:<br>
> > > ><br>
> > > ><br>
> > > weird debug output, you have logfilenr 5, then logfilenr 4 then again<br>
> > > logfilenr 5 ....<br>
> ><br>
> > Yes, indeed. Your existing code in master will alternate between those two<br>
> > files until the memory runs out. Which is because you are comparing the<br>
> > wrong two values. You compare the # that you ask for with the number that<br>
> > you want. You never parse (or use) the number that you got:<br>
> ><br>
><br>
> interesting, I think your Uemis is unique in that. even I cleaned up the<br>
> code, got<br>
> rid of the weird thing below I am not sure if my new patch can handle<br>
> alternating<br>
> logfilenr.<br>
<br>
</span>NO. Please read what I wrote. This happened because of a bug in your code.<br>
With my patches (that I haven't pushed) my Uemis is read just fine.<br></blockquote><div><br></div><div>I read what you wrote, the first code I installed, no changes from me, alternated between</div><div>two dives till the memory was full.</div><div>My code has a bug, indeed, and to explain would take a long time, hence I skip this till I</div><div>have more time - please trust me.</div><div><br></div><div>I implemented your suggestion and are at dive 200 now for importing, hence it works with mine</div><div>but the line where we decrement is never called but will most likely be called with your uemis. </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class=""><br>
> I will implement your suggestion below and see what it does to mine since<br>
> you<br>
> original code did alternate on my uemis between two dives till the memory<br>
> ran<br>
> out too - so the question is, is my Uemis f..u.. or your's :-)<br>
<br>
</span>I don't think so. Would you like me to push what I have so you can test<br>
it? Maybe that's easier then having you try to figure this out. It sounded<br>
earlier as if you were ready to submit a patch and I didn't want to step<br>
on your work, but if you haven't fixed this yet I can simply take my own<br>
fixes and have you work on top of that.<br></blockquote><div><br></div><div>please wait, I am sending the patch soon, I spend the last 2 hour or so testing it with all</div><div>my dives. With retry, with accepting, and mixed. </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class=""><br>
> the issue is that if we only decrement it will not work on my, so I need to<br>
> find some way to determine<br>
> whether the Uemis in use needs an increment or decrement - clearly funky<br>
> design by Uemis Zurich<br>
<br>
</span>No, it checks in either direction in my code. Why don't I just push my<br>
fixes. OK?<br></blockquote><div><br></div><div>where is my learning ? :-) you'll have my patch soon. </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<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></div>