> The discussion on this was in March 2013. This is it, I think:
http://apache-flex-development.2333347.n4.nabble.com/ArrayList-itemUpdateHandler-Change-td7774.html It looks to me like the intention of the person who originally created the new ArrayList (Michael A. Labriola) was for it to be tested with the rest of the framework and then to replace the original ArrayList: "Well, first thing I think is that we make sure everything still internally works in the framework. At some point though we will likely need to document this and bit the bullet. As someone who has been intimate with the framework for years I can guarantee you that the order of events in something like combobox has basically changed every other versions since flex 2, so we have precedent :) " However, I think the simplest thing for me to do would be to patch both classes with the Proxy stuff. At least that way I will not be making the framework *more* confusing. Gareth On 14 May 2015 at 22:30, Harbs <harbs.li...@gmail.com> wrote: > The discussion on this was in March 2013. > > On May 14, 2015, at 11:03 PM, Alex Harui <aha...@adobe.com> wrote: > > > I just looked quickly at the differences. It appears that the second > > ArrayList has different update event logic so it may be that it was safer > > to create a different ArrayList so we wouldn’t have to worry about > > backward compatibility issues. > > > > -Alex > > > > On 5/13/15, 2:31 PM, "Gareth Smith" <garethdanielsm...@gmail.com> wrote: > > > >> It looks like the second ArrayList was introduced by Justin Mclean in > >> this commit: > >> > >> > https://github.com/apache/flex-sdk/commit/621e4debe8cd46e9ec5a197256dc2b6e > >> 32fc47c3 > >> > >> I can't figure out why it was included though. Justin? > >> > >> Thanks > >> Gareth > >> > >> On 10/05/15 11:29, Gareth Smith wrote: > >>> Hi, > >>> > >>> I am putting together a patch for ArrayList to support square-bracket > >>> indexing and for-each loops, see: > >>> https://issues.apache.org/jira/browse/FLEX-15159 > >>> > >>> I am confused by the existence of not one, but two ArrayList classes: > >>> the "original" one at mx.collections.ArrayList and another one at > >>> org.apache.flex.collections.ArrayList. > >>> > >>> Why are there two classes? Looking at their diff, I would guess that > >>> mx.collections.ArrayList is newer: it has some additional > >>> optimisations, and an additional toJSON() method. > >>> > >>> I could patch both classes, but I would rather not introduce more > >>> duplication that might become a maintenance burden along the line. > >>> > >>> Thanks > >>> Gareth > >> > > > >
