<div dir="rtl"><div dir="ltr">Alright, that fixed the unit test!<br></div><div dir="ltr">No need to revert anything, from my side :)<br><br></div><div dir="ltr">Regards<br></div></div><div class="gmail_extra"><div dir="ltr"><br><div class="gmail_quote">2017-12-25 17:58 GMT+03:00 Safa Alfulaij <span dir="ltr"><<a href="mailto:safa1996alfulaij@gmail.com" target="_blank">safa1996alfulaij@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 .8ex;border-left:1px #ccc solid;border-right:1px #ccc solid;padding-left:1ex;padding-right:1ex"><div dir="rtl"><div dir="ltr">Maybe the problem is the sequence of setting beginCursor.<br></div><div dir="ltr">Maybe moving <a href="https://cgit.kde.org/ktexteditor.git/tree/src/document/katedocument.cpp#n3203" target="_blank">https://cgit.kde.org/<wbr>ktexteditor.git/tree/src/<wbr>document/katedocument.cpp#<wbr>n3203</a> to be before the <span style="font-family:monospace,monospace">if (!isValidTextPosition(<wbr>beginCursor))</span> statment will solve the problem, but not really sure and can't test.<br></div><div dir="ltr">I don't see any other difference than this.<br></div></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><div dir="ltr"><br><div class="gmail_quote">2017-12-25 17:44 GMT+03:00 Safa Alfulaij <span dir="ltr"><<a href="mailto:safa1996alfulaij@gmail.com" target="_blank">safa1996alfulaij@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 .8ex;border-left:1px #ccc solid;border-right:1px #ccc solid;padding-left:1ex;padding-right:1ex"><div dir="rtl"><div dir="ltr">Hi<br><br></div><div dir="ltr">Please revert the patch as I don't have more time to investigate this, sorry.<br></div><div dir="ltr">My system isn't ready for such testing and I need time to fix it (most probably in the mid of January)<br><br></div><div dir="ltr">Sorry for all the mess.<br><br></div><div dir="ltr">Regards<br></div></div><div class="m_-3759041980771470902HOEnZb"><div class="m_-3759041980771470902h5"><div class="gmail_extra"><br><div class="gmail_quote"><div dir="ltr">2017-12-25 12:59 GMT+03:00 Dominik Haumann <span dir="ltr"><<a href="mailto:dhaumann@kde.org" target="_blank">dhaumann@kde.org</a>></span>:</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="auto">Hi Safa,<div dir="auto"><br></div><div dir="auto">Looking at <a href="https://build.kde.org" target="_blank">https://build.kde.org</a>, ktexteditor is yellow since your change. Could you have a look at the unit test again?</div><div dir="auto"><br></div><div dir="auto">Greetings</div><span class="m_-3759041980771470902m_-6025747252073521362HOEnZb"><font color="#888888"><div dir="auto">Dominik</div></font></span></div>
</blockquote></div><br></div>
</div></div></blockquote></div></div></div>
</div></div></blockquote></div></div></div>