Don't apologize for not having time to look at this, I should have been able to work it out myself. However, I had missed that solution and do appreciate the help. My knowledge of how pointers and weak pointers etc. are kept and maintained in python is still not to the my standard of knowledge for c++. The bug has now been fixed and committed.

Thanks!
Luke


________________________________
From: John Haddon [notifications@github.com]
Sent: Saturday, April 06, 2013 11:13 AM
To: ImageEngine/gaffer
Cc: Luke Goddard
Subject: Re: [gaffer] MultiSelectionMenu (#57)


In python/GafferUI/MultiSelectionMenu.py:

> +class MultiSelectionMenu( GafferUI.Button ) :
> +
> +     __palette = None
> +
> +     def __init__(
> +             self,
> +             allowMultipleSelection = False,
> +             allowEmptySelection = True,
> +             **kw
> +     ) :
> +             self.__allowMultipleSelection = allowMultipleSelection
> +             self.__allowEmptySelection = allowEmptySelection
> +             self.__selectionChangedSignal = None
> +
> +             GafferUI.Button.__init__( self, **kw )
> +             self._menu = GafferUI.Menu( self.__addMenuDefinition, self._qtWidget() )


Sorry I didn't have time to getting round to hunting the cause of the test failures yesterday. I reckon this line here is probably your problem though. self references menu and menu references self (via self.__addMenuDefinition).

—
Reply to this email directly or view it on GitHub<https://github.com/ImageEngine/gaffer/pull/57/files#r3684550>.