- User Since
- Sep 24 2018, 1:53 PM (21 w, 2 d)
Wed, Jan 23
Thanks for the help! Didn't find the proper way of doing it with Ctrl+F.
It's ok that the change won't land in prod: release build doesn't have the crash, since it comes from Q_ASSERT, after which the value is properly clamped.
Dec 29 2018
Seems safe enough. It would just make debug build behave like the release one because of the same transformations made in loc() after asserts.
Dec 23 2018
All right, full bounds check it is. I could also reproduce y=-1 for the resized window with overscroll (wasn't resizing the window earlier).
Couldn't update the diff in this ticket. The corrected version is at https://phabricator.kde.org/D17757
This happens because loc() expects character-space indices, while selection-space indices are provided. The semantic of selection-space is that the index of the character after the last one select the last character.
Oct 22 2018
@hindenburg it's email@example.com.