I'm always sure, but thanks for your paranoia. On Tue, 10 Dec 2013 16:35:01 +0000 Tom Hacohen <tom.haco...@samsung.com> wrote:
> Just making sure it's correct, as it looks a bit odd: > You just removed a variable set, you didn't set it otherwise. I'm > assuming it was set twice (again later, when meant to), so it's fine, > but just raising the flag to make sure. > > -- > Tom. > > On 10/12/13 16:31, Mike Blumenkrantz wrote: > > discomfitor pushed a commit to branch master. > > > > http://git.enlightenment.org/core/enlightenment.git/commit/?id=09c3cd4ba9a7721af08c11dfb564957bbc1f8c84 > > > > commit 09c3cd4ba9a7721af08c11dfb564957bbc1f8c84 > > Author: Mike Blumenkrantz <zm...@samsung.com> > > Date: Tue Dec 10 11:03:00 2013 -0500 > > > > only set bd->destroyed when window is actually destroyed > > > > T617.1 > > --- > > src/bin/e_border.c | 2 -- > > 1 file changed, 2 deletions(-) > > > > diff --git a/src/bin/e_border.c b/src/bin/e_border.c > > index 00ff2a7..81baba8 100644 > > --- a/src/bin/e_border.c > > +++ b/src/bin/e_border.c > > @@ -5139,8 +5139,6 @@ _e_border_cb_window_hide(void *data __UNUSED__, > > { > > // printf(" hide2\n"); > > > > - if (bd->delete_requested) > > - bd->destroyed = 1; > > e_border_hide(bd, 0); > > e_object_del(E_OBJECT(bd)); > > } > > > ------------------------------------------------------------------------------ Rapidly troubleshoot problems before they affect your business. Most IT organizations don't have a clear picture of how application performance affects their revenue. With AppDynamics, you get 100% visibility into your Java,.NET, & PHP application. Start your 15-day FREE TRIAL of AppDynamics Pro! http://pubads.g.doubleclick.net/gampad/clk?id=84349831&iu=/4140/ostg.clktrk _______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
