From 2df013931bbb57f00b66cbaa115a846327ac2aea Mon Sep 17 00:00:00 2001 From: Albrecht Schlosser Date: Wed, 8 Jan 2020 19:50:35 +0100 Subject: Improve X11 coordinate clipping for text (STR 2798) Text outside the 16-bit X11 coordinate space must be clipped before calling X11 draw functions, otherwise text might appear anywhere in the window because X11 would "truncate" the coordinates instead of proper clipping (X11 handles only 16-bit coordinates). --- src/drivers/Xlib/Fl_Xlib_Graphics_Driver_font_xft.cxx | 13 +++++++++++-- 1 file changed, 11 insertions(+), 2 deletions(-) (limited to 'src/drivers/Xlib/Fl_Xlib_Graphics_Driver_font_xft.cxx') diff --git a/src/drivers/Xlib/Fl_Xlib_Graphics_Driver_font_xft.cxx b/src/drivers/Xlib/Fl_Xlib_Graphics_Driver_font_xft.cxx index 4fdd070aa..26d624b8d 100644 --- a/src/drivers/Xlib/Fl_Xlib_Graphics_Driver_font_xft.cxx +++ b/src/drivers/Xlib/Fl_Xlib_Graphics_Driver_font_xft.cxx @@ -782,6 +782,15 @@ void Fl_Xlib_Graphics_Driver::text_extents_unscaled(const char *c, int n, int &d } void Fl_Xlib_Graphics_Driver::draw_unscaled(const char *str, int n, int x, int y) { + + // transform coordinates and clip if outside 16-bit space (STR 2798) + + int x1 = x + offset_x_ * scale() + line_delta_; + if (x1 < clip_min() || x1 > clip_max()) return; + + int y1 = y + offset_y_ * scale() + line_delta_; + if (y1 < clip_min() || y1 > clip_max()) return; + #if USE_OVERLAY XftDraw*& draw_ = fl_overlay ? draw_overlay : ::draw_; if (fl_overlay) { @@ -814,9 +823,9 @@ void Fl_Xlib_Graphics_Driver::draw_unscaled(const char *str, int n, int x, int y const wchar_t *buffer = utf8reformat(str, n); #ifdef __CYGWIN__ - XftDrawString16(draw_, &color, ((Fl_Xlib_Font_Descriptor*)font_descriptor())->font, x+offset_x_*scale()+line_delta_, y+offset_y_*scale()+line_delta_, (XftChar16 *)buffer, n); + XftDrawString16(draw_, &color, ((Fl_Xlib_Font_Descriptor*)font_descriptor())->font, x1, y1, (XftChar16 *)buffer, n); #else - XftDrawString32(draw_, &color, ((Fl_Xlib_Font_Descriptor*)font_descriptor())->font, x+offset_x_*scale()+line_delta_, y+offset_y_*scale()+line_delta_, (XftChar32 *)buffer, n); + XftDrawString32(draw_, &color, ((Fl_Xlib_Font_Descriptor*)font_descriptor())->font, x1, y1, (XftChar32 *)buffer, n); #endif } } -- cgit v1.2.3