<div dir="ltr"><div>Hi Michael, Bjoern</div><div><br></div>On Tue, Jun 4, 2013 at 11:52 PM, Michael Stahl <span dir="ltr"><<a href="mailto:mstahl@redhat.com" target="_blank">mstahl@redhat.com</a>></span> wrote:<br>
<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">hi Tomaž,<br><br>i noticed that SwAutoCompleteWord appears to contain 2 different<br>


auto-complete mechanisms - an old vector m_WordList and a new<br>m_LookupTree that was introduced a year ago; is there some<br>ongoing work here?  can the old vector be removed?<br></blockquote><div> </div><div>
Ah yes this is still a to-do. Currently you need both because LookupTree does not implement an iteration through all words. You however need this for the dialog - so a temporary solution then was to just use both for the time being and put new words in both lists. I have forgot about this but now you reminded me, I will take a look and implement iteration.</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">regards,<br> michael</blockquote><div class="gmail_extra">


<br><br><div class="gmail_quote">On Wed, Jun 5, 2013 at 12:15 AM, Bjoern Michaelsen <span dir="ltr"><<a href="mailto:bjoern.michaelsen@canonical.com" target="_blank">bjoern.michaelsen@canonical.com</a>></span> wrote:<br>


<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">Hi there,<br>
<br>
sorry for shamelessly piggybacking:<br>
<div><div><br>
On Tue, Jun 04, 2013 at 11:52:49PM +0200, Michael Stahl wrote:<br>
> hi Tomaž,<br>
><br>
> i noticed that SwAutoCompleteWord appears to contain 2 different<br>
> auto-complete mechanisms - an old vector m_WordList and a new<br>
> m_LookupTree that was introduced a year ago; is there some<br>
> ongoing work here?  can the old vector be removed?<br>
<br>
</div></div>Oh, while we are at it:<br>
<br>
This fancy new lookup tree seems to cause this crasher/stacktrace:<br>
<br>
 <a href="https://bugs.launchpad.net/ubuntu/+source/libreoffice/+bug/1162191" target="_blank">https://bugs.launchpad.net/ubuntu/+source/libreoffice/+bug/1162191</a><br>
<br>
since LibreOffice 4.0~beta2, which was the first 4.0 release packaged for<br>
Ubuntu, causing some ~50 crashes on Ubuntu daily (second most common stacktrace<br>
for libreoffice right now).<br></blockquote><div><br></div><div>This is pretty bad! There was already a bug report about this [1] but was later closed.. so I did not look further into this. I will do a review of the implementation, maybe I will spot the problem. </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">

Best,<br>
<br>
Bjoern<br>
</blockquote></div><br></div><div class="gmail_extra" style>Regards, Tomaž</div><div class="gmail_extra" style><br></div><div class="gmail_extra" style>[1] <a href="https://bugs.freedesktop.org/show_bug.cgi?id=55315">https://bugs.freedesktop.org/show_bug.cgi?id=55315</a></div>

<div class="gmail_extra"><br></div></div>