I applied them all except #7 and #18. #7 looks good, but I don’t have an easy way to test it. The chances of the code breaking something seems very low, but if someone can somehow test it, that would be preferable.
I commented on #18. I’m not sure about the benefits of that one… On Nov 18, 2015, at 8:07 AM, Justin Mclean <[email protected]> wrote: > Hi, > > We also have several outstanding pull requests. These look reasonable to me: > https://github.com/apache/flex-sdk/pull/20 > <https://github.com/apache/flex-sdk/pull/20> > https://github.com/apache/flex-sdk/pull/16 > <https://github.com/apache/flex-sdk/pull/16> > https://github.com/apache/flex-sdk/pull/12 > <https://github.com/apache/flex-sdk/pull/12> > https://github.com/apache/flex-sdk/pull/8 > <https://github.com/apache/flex-sdk/pull/8> > https://github.com/apache/flex-sdk/pull/7 > <https://github.com/apache/flex-sdk/pull/7> > > Not sure about the performance implications of this one: > https://github.com/apache/flex-sdk/pull/18 > <https://github.com/apache/flex-sdk/pull/18> > > Anyone have time to merge and test? > > Thanks, > Justin
