CanvasTarget _refresh method takes an arbitrary number of arguments, so

it doesn't need to be wrapped when passed as an event callback

fsl/fslview/gl/__init__.py

@@ -237,7 +237,7 @@ class OSMesaCanvasTarget(object):
                                  self._height)
 
         
-    def _refresh(self):
+    def _refresh(self, *a):
         """Does nothing. This canvas is for static (i.e. unchanging) rendering.
         """
         pass
@@ -370,7 +370,7 @@ class WXGLCanvasTarget(object):
         return True
 
         
-    def _refresh(self):
+    def _refresh(self, *a):
         """Triggers a redraw via the :mod:`wx` `Refresh` method."""
         self.Refresh()
 

fsl/fslview/gl/slicecanvas.py

@@ -387,20 +387,18 @@ class SliceCanvas(props.HasProperties):
                 self._refresh()
             genGLObject()
                 
-            def refresh(*a): self._refresh()
-
-            image  .addListener('data',          self.name, refresh)
+            image  .addListener('data',          self.name, self._refresh)
             display.addListener('imageType',     self.name, genGLObject)
-            display.addListener('enabled',       self.name, refresh)
-            display.addListener('transform',     self.name, refresh)
-            display.addListener('interpolation', self.name, refresh)
-            display.addListener('alpha',         self.name, refresh)
-            display.addListener('displayRange',  self.name, refresh)
-            display.addListener('clipLow',       self.name, refresh)
-            display.addListener('clipHigh',      self.name, refresh)
-            display.addListener('resolution',    self.name, refresh)
-            display.addListener('cmap',          self.name, refresh)
-            display.addListener('volume',        self.name, refresh)
+            display.addListener('enabled',       self.name, self._refresh)
+            display.addListener('transform',     self.name, self._refresh)
+            display.addListener('interpolation', self.name, self._refresh)
+            display.addListener('alpha',         self.name, self._refresh)
+            display.addListener('displayRange',  self.name, self._refresh)
+            display.addListener('clipLow',       self.name, self._refresh)
+            display.addListener('clipHigh',      self.name, self._refresh)
+            display.addListener('resolution',    self.name, self._refresh)
+            display.addListener('cmap',          self.name, self._refresh)
+            display.addListener('volume',        self.name, self._refresh)
 
         self._refresh()