Ok, as many people as stated they will prefer not revert, I will not make any changes and will keep the IList removeItem new method and the method implementations. I will postpone a Discussion thread for a later time when we have more use cases that will make us to talk and discuss about global philosophy.
Thanks all. Carlos 2013/3/25 Cosma Colanicchia <[email protected]> > +1 on adding the method to IList... I have *lot* of code that look like > > var index:int = list.getItemIndex(object); > if (index >= 0) { > list.removeItemAt(index); > } > > I think this is ugly and verbose, and it pollutes my function namespace - I > ends up with "indexOfItemToRemove1" and some like this, as AS3 lacks > java-like block-level variables. Having static helper functions to mitigate > this kind of micro-issues is not ideal, as it makes the code less readable > and fluent, at least for my eyes. > > I also think it is actually correct that existing methods are broken on > first compile: its a nice way for owners of that code to be aware of the > change, and thus look up if their custom method implementation can be > trashed or must be kept and renamed for some reason. > > > > > > 2013/3/25 Alex Harui <[email protected]> > > > > > > > > > On 3/25/13 8:00 AM, "Carlos Rovira" <[email protected]> > wrote: > > > > > So Alex, > > > > > > what we do? revert? let the change live? make a Discuss thread so > people > > > could express the phylosophy with this issue and future similar > tickets? > > > > > > Please, let us know so we can act accordingly > > > > > It is up to you. I have stated my concerns. There is no right or wrong > > answer. If you are willing to take the time to start a discuss or poll, > > feel free. So far, four people have voiced opinions and nobody thinks > you > > must revert. > > > > -- > > Alex Harui > > Flex SDK Team > > Adobe Systems, Inc. > > http://blogs.adobe.com/aharui > > > > > -- Carlos Rovira Director de Tecnología M: +34 607 22 60 05 F: +34 912 94 80 80 http://www.codeoscopic.com http://www.directwriter.es http://www.avant2.es
