Hello again, So I looked at the 1.7 branch and the code there is very different so the bug wasn't in it. The trunk uses a hash table with a free function that frees the Part_Lookup structure, and the bug was that it was manually freeing one item without removing it from the hash table. In 1.7, it's actually using a list, and the hashtable is created with eina_hash_string_superfast_new (so, no free func) and it actually removes it from that hash table when it frees it. Basically, the bug in trunk was a side-effect/remnant from the refactoring of the code from the branch.. so there is no need to backport it.
On Wed, Nov 28, 2012 at 1:46 AM, Youness Alaoui < [email protected]> wrote: > Oh cool, I'll backport it tomorrow! Thanks! > I never realized there were branches, I thought new releases were always > taken from the trunk! > > Thanks for the answers! > > On Tue, Nov 27, 2012 at 9:59 PM, Carsten Haitzler <[email protected]>wrote: > >> On Tue, 27 Nov 2012 11:45:20 -0500 Youness Alaoui >> <[email protected]> said: >> >> > Backported? Sorry, what do you mean ? Are there different development >> > branches now ? (I haven't paid much attention to E develpment lately) >> > If yes, then yeah, it should be backported as it's a critical fix in my >> > opinion. >> >> look in branches (not in trunk)... we have a branch per stable release of >> efl >> where we backport to.. thats how 1.7.1 and 1.7.2 come out.. from the >> stable >> branch. we've done this since 1.0 :) not new. :) >> >> > On Mon, Nov 26, 2012 at 4:26 AM, Cedric BAIL <[email protected]> >> wrote: >> > >> > > On Sat, Nov 24, 2012 at 8:15 AM, Enlightenment SVN >> > > <[email protected]> wrote: >> > > > Log: >> > > > edje: Fix segfault when deleted part stays in the hash table >> > > >> > > Shouldn't that be backported ? >> > > -- >> > > Cedric BAIL >> > > >> > > >> > > >> ------------------------------------------------------------------------------ >> > > Monitor your physical, virtual and cloud infrastructure from a single >> > > web console. Get in-depth insight into apps, servers, databases, >> vmware, >> > > SAP, cloud infrastructure, etc. Download 30-day Free Trial. >> > > Pricing starts from $795 for 25 servers or applications! >> > > http://p.sf.net/sfu/zoho_dev2dev_nov >> > > _______________________________________________ >> > > enlightenment-devel mailing list >> > > [email protected] >> > > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel >> > > >> > >> ------------------------------------------------------------------------------ >> > Monitor your physical, virtual and cloud infrastructure from a single >> > web console. Get in-depth insight into apps, servers, databases, vmware, >> > SAP, cloud infrastructure, etc. Download 30-day Free Trial. >> > Pricing starts from $795 for 25 servers or applications! >> > http://p.sf.net/sfu/zoho_dev2dev_nov >> > _______________________________________________ >> > enlightenment-devel mailing list >> > [email protected] >> > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel >> > >> >> >> -- >> ------------- Codito, ergo sum - "I code, therefore I am" -------------- >> The Rasterman (Carsten Haitzler) [email protected] >> >> > ------------------------------------------------------------------------------ Keep yourself connected to Go Parallel: INSIGHTS What's next for parallel hardware, programming and related areas? Interviews and blogs by thought leaders keep you ahead of the curve. http://goparallel.sourceforge.net _______________________________________________ enlightenment-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
