<div dir="ltr"><div><div><div>Colomban,<br><br></div>Really my fault (<br></div>Thanks a lot for participation and detecting root cause! I've tested your patch, it fixes the issue.<br></div><div>Thanks again!<br><br></div><div>P.S. The only question, when the new version of geany-plugins is planned to be released?<br></div><div><br></div><br><div class="gmail_extra"><br><div class="gmail_quote">On 9 March 2015 at 18:19, Colomban Wendling <span dir="ltr"><<a href="mailto:lists.ban@herbesfolles.org" target="_blank">lists.ban@herbesfolles.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
tl;dr: I found the issue in the end, which is in a call from the plugin.<br>
 Here below is my train of responses because they still are of some<br>
interest, but you can skip directly to the last paragraph.<br>
<br>
Le 09/03/2015 14:48, Volodymyr Kononenko a écrit :<br>
<span class="">> A search with git bisect has shown, that the bug was introduced with<br>
> updating Scintilla to 3.3.2:<br>
><br>
</span>> […]<br>
><br>
> Similar issue <<a href="http://sourceforge.net/p/scintilla/bugs/1604/" target="_blank">http://sourceforge.net/p/scintilla/bugs/1604/</a>> I've found<br>
<span class="">> on Scintilla bug tracker was fixed only for win32 - here is the patch<br>
</span>> <<a href="http://sourceforge.net/p/scintilla/code/ci/23f89aef53c7ab73bdd45ae1302b97698682e3cd" target="_blank">http://sourceforge.net/p/scintilla/code/ci/23f89aef53c7ab73bdd45ae1302b97698682e3cd</a>>.<br>
<br>
I highly doubt it's anything like the same issue, because the one<br>
described here is a redraw issue, which should get fixed when triggering<br>
another redraw, like e.g. scrolling the offending part offscreen and<br>
back, while the issues I see with pairtaghighlighter persist after<br>
redraws (which means the indicators themselves are still here).<br>
<br>
Moreover, clearing the indicators on the whole buffer:<br>
<br>
> scintilla_send_message(sci, SCI_INDICATORCLEARRANGE, 0, sci_get_length(sci));<br>
<br>
instead of only the range passed to the function fixes the issue for me,<br>
so I would really bet on inappropriate clear range.<br>
<br>
Which is indeed kind of weird because it really does work with 1.23 and<br>
not 1.24, even with the very same plugin .so.<br>
<span class=""><br>
> I tried to detect the first Scintilla commit, which introduced the<br>
> issue, but I see that merging Scintilla to Geany sources is non-trivial.<br>
> Build failed after the first attempt.<br>
><br>
> Folks, what do you think about it? Is it enough information that it is<br>
> Scintilla issue and we need to report it or something was not taken into<br>
> account during upgrading to 3.3.2?<br>
<br>
</span>Actually it's the plugin's SCI_INDICATORCLEARRANGE<br>
 <<a href="http://www.scintilla.org/ScintillaDoc.html#SCI_INDICATORCLEARRANGE" target="_blank">http://www.scintilla.org/ScintillaDoc.html#SCI_INDICATORCLEARRANGE</a>><br>
that is incorrect: the second parameter should be the *length* to clear,<br>
not the end position.  Apparently earlier Scintilla versions were<br>
forgiving on the range, but new ones aren't and do nothing with invalid<br>
ranges.<br>
<br>
I made you a PR fixing the issue:<br>
<a href="https://github.com/geany/geany-plugins/pull/200" target="_blank">https://github.com/geany/geany-plugins/pull/200</a><br>
<br>
<br>
Regards,<br>
Colomban<br>
<div class="HOEnZb"><div class="h5">_______________________________________________<br>
Devel mailing list<br>
<a href="mailto:Devel@lists.geany.org">Devel@lists.geany.org</a><br>
<a href="https://lists.geany.org/cgi-bin/mailman/listinfo/devel" target="_blank">https://lists.geany.org/cgi-bin/mailman/listinfo/devel</a><br>
</div></div></blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature">Best regards,<br>Volodymyr Kononenko<br><a href="http://kononenko.ws" target="_blank">http://kononenko.ws</a><br></div>
</div></div>