Use UnmodifiableMapView in compiler

sdk/lib/_internal/compiler/implementation/mirrors/mirrors_util.dart

 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 library mirrors_util;
 
 import 'dart:collection' show Queue, IterableBase;
 
 import 'source_mirrors.dart';
 
@@ skipping 162 matching lines @@
 }
 
 Iterable<DeclarationMirror> membersOf(
     Map<Symbol, DeclarationMirror> declarations) {
   return declarations.values.where(
       (mirror) => mirror is MethodMirror || mirror is VariableMirror);
 }
 
 Iterable<TypeMirror> classesOf(
     Map<Symbol, DeclarationMirror> declarations) {
-  return declarations.values.where((mirror) => mirror is ClassMirror);
+  return new _TypeOfIterable<ClassMirror>(declarations.values);
 }
 
 Iterable<TypeMirror> typesOf(
     Map<Symbol, DeclarationMirror> declarations) {
-  return declarations.values.where((mirror) => mirror is TypeMirror);
+  return new _TypeOfIterable<TypeMirror>(declarations.values);
 }
 
 Iterable<MethodMirror> methodsOf(
     Map<Symbol, DeclarationMirror> declarations) {
-  return declarations.values.where(
-      (mirror) => mirror is MethodMirror && mirror.isRegularMethod);
+  return _anyMethodOf(declarations).where((mirror) => mirror.isRegularMethod);
 }
 
 Iterable<MethodMirror> constructorsOf(
     Map<Symbol, DeclarationMirror> declarations) {
-  return declarations.values.where(
-      (mirror) => mirror is MethodMirror && mirror.isConstructor);
+  return _anyMethodOf(declarations).where((mirror) => mirror.isConstructor);
 }
 
 Iterable<MethodMirror> settersOf(
     Map<Symbol, DeclarationMirror> declarations) {
-  return declarations.values.where(
-      (mirror) => mirror is MethodMirror && mirror.isSetter);
+  return _anyMethodOf(declarations).where((mirror) => mirror.isSetter);
 }
 
 Iterable<MethodMirror> gettersOf(
     Map<Symbol, DeclarationMirror> declarations) {
-  return declarations.values.where(
-      (mirror) => mirror is MethodMirror && mirror.isGetter);
+  return _anyMethodOf(declarations).where((mirror) => mirror.isGetter);
 }
 
+Iterable<MethodMirror> _anyMethodOf(

[herhut, 2014/04/28 08:24:41]

Maybe expose this (docgen seems to be a good use case).

[kevmoo, 2014/04/28 09:23:01]

Done.

+    Map<Symbol, DeclarationMirror> declarations) =>

[herhut, 2014/04/28 08:24:41]

Nit: Do not use => across line breaks, use {} instead.

[kevmoo, 2014/04/28 09:23:01]

Done.

+        new _TypeOfIterable<MethodMirror>(declarations.values);
+
 Iterable<VariableMirror> variablesOf(
     Map<Symbol, DeclarationMirror> declarations) {
-  return declarations.values.where((mirror) => mirror is VariableMirror);
+  return new _TypeOfIterable<VariableMirror>(declarations.values);
 }
 
+class _TypeOfIterable<TTarget> extends IterableBase<TTarget> {

[herhut, 2014/04/28 08:24:41]

I prefer T for type variables.

[kevmoo, 2014/04/28 09:23:01]

Done.

+  final Iterable _source;
 
+  _TypeOfIterable(this._source);
+
+  Iterator<TTarget> get iterator =>

[herhut, 2014/04/28 08:24:41]

Nit: Do not use => across line breaks.

[kevmoo, 2014/04/28 09:23:01]

Done.

+      new _TypeOfIterator<TTarget>(_source.iterator);
+}
+
+class _TypeOfIterator<TTarget> implements Iterator<TTarget> {
+  final Iterator _source;
+
+  TTarget get current => _source.current as TTarget;

[herhut, 2014/04/28 08:24:41]

Do you really need an as check here?

[kevmoo, 2014/04/28 09:23:01]

Done.

+
+  _TypeOfIterator(this._source);
+
+  bool moveNext() {
+    while(_source.moveNext()) {
+      if (_source.current is TTarget) {
+        return true;
+      }
+    }
+    return false;
+  }
+}
 
 bool isObject(TypeMirror mirror) =>
     mirror is ClassMirror && mirror.superclass == null;
 
 /// Returns `true` if [cls] is declared in a private dart library.
 bool isFromPrivateDartLibrary(ClassMirror cls) {
   if (isMixinApplication(cls)) cls = cls.mixin;
   var uri = getLibrary(cls).uri;
   return uri.scheme == 'dart' && uri.path.startsWith('_');
 }
@@ skipping 165 matching lines @@
     // Try type variables.
     result = mirror.typeVariables.firstWhere(
         (TypeVariableMirror v) => v.simpleName == id, orElse: () => null);
   } else if (mirror is MethodMirror) {
     result = mirror.parameters.firstWhere(
         (ParameterMirror p) => p.simpleName == id, orElse: () => null);
   }
   return result;
 
 }