[Sugar-devel] [PATCH sugar 18/20] Get rid of the bootstrap function
Daniel Narvaez
dwnarvaez at gmail.com
Mon Dec 10 19:39:43 EST 2012
Stupid mistake in this one caused the code to be executed multiple
times. Sent an updated patch.
On 10 December 2012 20:11, Daniel Narvaez <dwnarvaez at gmail.com> wrote:
> From: Daniel Narvaez <dwnarvaez at gmail.com>
>
> The name doesn't say anything about the kind of initialization it
> should host, hard to say if part of this stuff should actually
> be in main or the other way around.
>
> Keeping it in the check_window_manager at least tells us when
> these initializations are run.
> ---
> src/jarabe/main.py | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/src/jarabe/main.py b/src/jarabe/main.py
> index 02e0011..2ff06f9 100755
> --- a/src/jarabe/main.py
> +++ b/src/jarabe/main.py
> @@ -91,9 +91,7 @@ def _check_for_window_manager(screen):
> wm_name = screen.get_window_manager_name()
> if wm_name is not None:
> screen.disconnect_by_func(_window_manager_changed_cb)
> - _bootstrap()
>
> -def _bootstrap():
> _setup_window_manager()
>
> launcher.setup()
> --
> 1.7.10.4
>
--
Daniel Narvaez
More information about the Sugar-devel
mailing list