Le 10/03/2017 à 14:59, Jürgen Spitzmüller a écrit :
I have no clear idea either (since you asked me). The code was written
by Abdel, I just copied it over from LayoutBox. But I try:

OK, I see now.

AFAICS it can actually be -1, if the filter string is not found. So we
probably should not assert here, but leave the loop.

I suspect that we now that the character is there, because we have filtered the elements with a regex that limits to them.

- LASSERT(p != -1, /**/);
+ if (p < 0)
+     break;

Or do a continue?

OK, I'll do a simple cleanup, it looks like this deserve a later plastic surgery. In particular, using a QTextDocument element for each item seems a bit exaggerated.

JMarc

Reply via email to