From 0521234fec3de6f0cd98b1d666cdf351b72f7b48 Mon Sep 17 00:00:00 2001 From: Sascha Silbe Date: Fri, 15 Oct 2010 20:11:10 +0000 Subject: [PATCH] CellRendererInvoker._redraw_path(): assure column is defined We know that our code is correct and column should always be assigned, but it never hurts to act defensively and guard against bugs in other pieces of the code (or other components like GTK). Reviewed-by: James Cameron Acked-by: Simon Schampijer CC: Aleksey Lim --- src/sugar/graphics/palettewindow.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/sugar/graphics/palettewindow.py b/src/sugar/graphics/palettewindow.py index 991acc87..ee9be966 100644 --- a/src/sugar/graphics/palettewindow.py +++ b/src/sugar/graphics/palettewindow.py @@ -938,9 +938,11 @@ class CellRendererInvoker(Invoker): self.notify_mouse_leave() def _redraw_path(self, path): + column = None for column in self._tree_view.get_columns(): if self._cell_renderer in column.get_cell_renderers(): break + assert column is not None area = self._tree_view.get_background_area(path, column) x, y = \ self._tree_view.convert_bin_window_to_widget_coords(area.x, area.y)