if traceur compiled methods has function declarations, rather than anonymous function expressions, we could do a completely terse recursive version in one line for extra ambiguity

_buildList(tree:any):List<Type> {
  return ListWrapper.reduce((item, list) => ListWrapper.concat(item, ListWrapper.isList(list) ? _buildList(list) : list , ListWrapper.create())
}
// or shorthand 
_buildList(tree) {
  return tree.reduce((item, list) => item.concat(isList(list) ? _buildList(list) : list , []);
}
// or more descriptive 
_buildList: _buildList(tree) {
  return tree.reduce(function(item, memo) { 
    if (isList(memo)) {
      return _buildList(memo)
    } else {
      return item.concat(memo);
    }
  }, [])
}

@gdi2290 I think that is reasonable for JS, but due to way Dart works it is not possible. I am fine with supporting more things in JS and only subset in Dart, but this creates a least common denominator.

Perf cleanup, otherwise it looks good to me.

Thanks for the comments, cleanup done

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

_{This action has been performed automatically by a bot.}