On Wed, 28 Nov 2012 15:07:17 -0500 Youness Alaoui
<[email protected]> said:

> 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.

good answer... and reason not to backport :)

> 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]
> >>
> >>
> >


-- 
------------- 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

Reply via email to