[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