@techee commented on this pull request.


> +     if (sci_get_line_from_position(sci, pos_next) == line)
+               char_width = SSM(sci, SCI_POINTXFROMPOSITION, 0, pos_next) - x;

> I went ahead and implemented the more basic version that is similar to what 
> was there but don't produce incorrect results on the wrapping corner case. So 
> I'm happier, and you're not too unhappy because it's not as complicated as it 
> could be 

The best of both worlds indeed, LGTM :-)

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3316#discussion_r1358313969
You are receiving this because you are subscribed to this thread.

Message ID: <geany/geany/pull/3316/review/1676581...@github.com>

Reply via email to