From dbec27a4c7733b44e33ba898fef7e8636e86caa0 Mon Sep 17 00:00:00 2001 From: dkf Date: Thu, 13 Sep 2012 20:47:03 +0000 Subject: [Bug 3567453]: Clip regions must be cleared with XSetClipMask and not TkSetRegion on X11. --- ChangeLog | 9 +++++++++ generic/ttk/ttkEntry.c | 10 ++++++---- generic/ttk/ttkLabel.c | 4 ++-- xlib/xgc.c | 2 +- 4 files changed, 18 insertions(+), 7 deletions(-) diff --git a/ChangeLog b/ChangeLog index 54beeb3..f87abff 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,12 @@ +2012-09-13 Donal K. Fellows + + * generic/ttk/ttkEntry.c (EntryDisplay): [Bug 3567453]: Clip regions + * generic/ttk/ttkLabel.c (TextDraw): must be cleared with XSetClipMask + * xlib/xgc.c (TkSetRegion): and not TkSetRegion, or crashes will ensue + on X11-based builds, which can't handle None for a region argument. + Added a clean panic to the non-X11 TkSetRegion to catch this case and + stop such confusion from happening again. + 2012-09-11 Donal K. Fellows * generic/ttk/ttkEntry.c (EntryDisplay): [Bug 3566594]: Must manually diff --git a/generic/ttk/ttkEntry.c b/generic/ttk/ttkEntry.c index 22da65c..5c280e4 100644 --- a/generic/ttk/ttkEntry.c +++ b/generic/ttk/ttkEntry.c @@ -1151,7 +1151,9 @@ static GC EntryGetGC(Entry *entryPtr, Tcl_Obj *colorObj, TkRegion clip) mask |= GCForeground; } gc = Tk_GetGC(entryPtr->core.tkwin, mask, &gcValues); - TkSetRegion(Tk_Display(entryPtr->core.tkwin), gc, clip); + if (clip != None) { + TkSetRegion(Tk_Display(entryPtr->core.tkwin), gc, clip); + } return gc; } @@ -1254,7 +1256,7 @@ static void EntryDisplay(void *clientData, Drawable d) gc = EntryGetGC(entryPtr, es.insertColorObj, clipRegion); XFillRectangle(Tk_Display(tkwin), d, gc, cursorX-cursorWidth/2, cursorY, cursorWidth, cursorHeight); - TkSetRegion(Tk_Display(tkwin), gc, None); + XSetClipMask(Tk_Display(tkwin), gc, None); Tk_FreeGC(Tk_Display(tkwin), gc); } @@ -1265,7 +1267,7 @@ static void EntryDisplay(void *clientData, Drawable d) Tk_Display(tkwin), d, gc, entryPtr->entry.textLayout, entryPtr->entry.layoutX, entryPtr->entry.layoutY, leftIndex, rightIndex); - TkSetRegion(Tk_Display(tkwin), gc, None); + XSetClipMask(Tk_Display(tkwin), gc, None); Tk_FreeGC(Tk_Display(tkwin), gc); /* Overwrite the selected portion (if any) in the -selectforeground color: @@ -1276,7 +1278,7 @@ static void EntryDisplay(void *clientData, Drawable d) Tk_Display(tkwin), d, gc, entryPtr->entry.textLayout, entryPtr->entry.layoutX, entryPtr->entry.layoutY, selFirst, selLast); - TkSetRegion(Tk_Display(tkwin), gc, None); + XSetClipMask(Tk_Display(tkwin), gc, None); Tk_FreeGC(Tk_Display(tkwin), gc); } diff --git a/generic/ttk/ttkLabel.c b/generic/ttk/ttkLabel.c index 55bf36d..597682f 100644 --- a/generic/ttk/ttkLabel.c +++ b/generic/ttk/ttkLabel.c @@ -185,8 +185,8 @@ static void TextDraw(TextElement *text, Tk_Window tkwin, Drawable d, Ttk_Box b) #ifdef HAVE_XFT TkUnixSetXftClipRegion(None); #endif - TkSetRegion(Tk_Display(tkwin), gc1, None); - TkSetRegion(Tk_Display(tkwin), gc2, None); + XSetClipMask(Tk_Display(tkwin), gc1, None); + XSetClipMask(Tk_Display(tkwin), gc2, None); TkDestroyRegion(clipRegion); } Tk_FreeGC(Tk_Display(tkwin), gc1); diff --git a/xlib/xgc.c b/xlib/xgc.c index 6c5e31c..fcc2798 100644 --- a/xlib/xgc.c +++ b/xlib/xgc.c @@ -436,7 +436,7 @@ TkSetRegion( TkRegion r) { if (r == None) { - FreeClipMask(gc); + Tcl_Panic("must not pass None to TkSetRegion for compatibility with X11; use XSetClipMask instead"); } else { TkpClipMask *clip_mask = AllocClipMask(gc); -- cgit v0.12