From: Gustavo S. B. <bar...@pr...> - 2008-09-15 16:57:39
|
On Tue, Sep 9, 2008 at 9:34 PM, Gustavo Sverzut Barbieri <bar...@pr...> wrote: > On Tue, Sep 9, 2008 at 9:14 PM, Gustavo Sverzut Barbieri > <bar...@pr...> wrote: >> On Tue, Sep 9, 2008 at 8:41 PM, <no-...@en...> wrote: >>> Author: barbieri >>> Date: 2008-09-09 16:41:42 -0700 (Tue, 09 Sep 2008) >>> New Revision: 35908 >>> >>> Modified: >>> trunk/edje/src/lib/edje_calc.c trunk/edje/src/lib/edje_private.h trunk/edje/src/lib/edje_smart.c trunk/edje/src/lib/edje_util.c >>> Log: >>> >>> Revert edje calculate optimizations. >>> >>> Edje is tricky, it's event processing is too weird and Cedric's >>> changes to make it work are not working as expected. Edje freezes >>> itself while processing signals, but in mouse down cb it forces >>> recalculate, which seems was previously ignored, but now they are not. >>> >>> We should look at how to fix this and then re-apply this patch. >> >> Attached is a patch to attempt to fix the issues. It will not look for >> postponed flag before recalculate (postpone does not force it anymore) >> and for input event callbacks it will force recalculation with >> _edje_recalc_do(). >> >> I grep'ed for other _edje_recalc() code, there are some suspicious >> cases in edje_program(), it would be great if people with >> embryo-intensive applications could apply this patch and report if >> there are problems. > > As I suspected, edje_program.c code does need to forcelly call > _edje_recalc_do(). Attached are two patches, the preferred one that > just forces recalc after scripts (embryo) run and another that forces > it after all kind of programs run. Please try the preferred and if it > does not work try alternative. Please report which one worked or > failed. update of patch so it applies nicely to current svn. Nothing else changed. -- Gustavo Sverzut Barbieri http://profusion.mobi embedded systems -------------------------------------- MSN: bar...@gm... Skype: gsbarbieri Mobile: +55 (19) 9225-2202 |