On Wed, 21 Mar 2007 17:22:17 +0100 Cedric BAIL <[EMAIL PROTECTED]> babbled:

ok 2 patches in - edje font hash and tiler speedup - but the list sort in place
is not - the reason i removed it last time is that i found lists being sorted
were losing items. i spotted this in e17's efm code which did a lot of list
and mini-list and partial list sorting stuff (from memory). you might want to
go over that code again.

as for sdl engine and rfc for events and other things - i will get to them.
this blast of mails from you is the last major lump i need to deal with on the
e-devel list :)

> I have also some patch that are still waiting their way to the CVS.
> 
> - edje_font_hash.diff: Use Evas_Hash for font lookup.
>       After upgrading my ubuntu, I experienced the same bug raster
> experienced on his machine (all dialog box where empty). I was able to find
> it in _edje_font_is_embedded from edje_textblock_styles.c.
> 
> static int
> _edje_font_is_embedded(Edje_File *edf, char *font)
> {
>    Evas_List *l;
> 
>    if (!edf->font_dir) return 0;
>    for (l = edf->font_dir->entries; l; l = l->next)
>      {
>         Edje_Font_Directory_Entry *fnt = l->data;
> 
>         if ((fnt->entry) && (!strcmp(fnt->entry, font)))
>           return 1;
>      }
>    return 1;
> }
> 
>        I read this function too quickly, and didn't see that it really only
> check edf->font_dir. The rest was only here to slow every thing :) So now the
> patch must work correctly.
> 
> - evas_list_sort_inplace.diff: Replace the merge sort algorithm with an in 
> place merge sort.
>       I don't remember why it didn't get inside but I am using it in my own
> code without problem and it's fast. If I have time, I should port this to 
> ecore_list_sort.
> 
> - evas_tiler_speedup.diff: Walk progressively over the Tilebuf_Tile.
>       In some of my test I did notice that this function could take some
> time (It was with moving evas rectangle if I remember correctly). I basically
> took the costly operation out of the loop.
> 
> Cedric
> 


-- 
------------- Codito, ergo sum - "I code, therefore I am" --------------
The Rasterman (Carsten Haitzler)    [EMAIL PROTECTED]
裸好多
Tokyo, Japan (東京 日本)

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to