Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(66)

Unified Diff: pkg/compiler/lib/src/elements/elements.dart

Issue 1859343004: dartfmt pkg/compiler (Closed) Base URL: git@github.com:dart-lang/sdk.git@master
Patch Set: Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « pkg/compiler/lib/src/elements/common.dart ('k') | pkg/compiler/lib/src/elements/modelx.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: pkg/compiler/lib/src/elements/elements.dart
diff --git a/pkg/compiler/lib/src/elements/elements.dart b/pkg/compiler/lib/src/elements/elements.dart
index 2e70d1f31192e9f854bf0fe6077afcc864ed2ddf..715069c29c0761093eefce642df25bdd871e709a 100644
--- a/pkg/compiler/lib/src/elements/elements.dart
+++ b/pkg/compiler/lib/src/elements/elements.dart
@@ -5,25 +5,18 @@
library elements;
import '../common.dart';
-import '../common/resolution.dart' show
- Resolution;
-import '../compiler.dart' show
- Compiler;
+import '../common/resolution.dart' show Resolution;
+import '../compiler.dart' show Compiler;
import '../constants/constructors.dart';
import '../constants/expressions.dart';
-import '../core_types.dart' show
- CoreClasses;
+import '../core_types.dart' show CoreClasses;
import '../dart_types.dart';
-import '../resolution/scope.dart' show
- Scope;
-import '../resolution/tree_elements.dart' show
- TreeElements;
+import '../resolution/scope.dart' show Scope;
+import '../resolution/tree_elements.dart' show TreeElements;
import '../ordered_typeset.dart' show OrderedTypeSet;
import '../script.dart';
-import '../tokens/token.dart' show
- Token,
- isUserDefinableOperator,
- isMinusOperator;
+import '../tokens/token.dart'
+ show Token, isUserDefinableOperator, isMinusOperator;
import '../tree/tree.dart';
import '../util/characters.dart' show $_;
import '../util/util.dart';
@@ -430,9 +423,8 @@ class Elements {
}
/// Unwraps [element] reporting any warnings attached to it, if any.
- static Element unwrap(Element element,
- DiagnosticReporter listener,
- Spannable spannable) {
+ static Element unwrap(
+ Element element, DiagnosticReporter listener, Spannable spannable) {
if (element != null && element.isWarnOnUse) {
WarnOnUseElement wrappedElement = element;
element = wrappedElement.unwrap(listener, spannable);
@@ -450,11 +442,11 @@ class Elements {
}
static bool isInstanceField(Element element) {
- return !Elements.isUnresolved(element)
- && element.isInstanceMember
- && (identical(element.kind, ElementKind.FIELD)
- || identical(element.kind, ElementKind.GETTER)
- || identical(element.kind, ElementKind.SETTER));
+ return !Elements.isUnresolved(element) &&
+ element.isInstanceMember &&
+ (identical(element.kind, ElementKind.FIELD) ||
+ identical(element.kind, ElementKind.GETTER) ||
+ identical(element.kind, ElementKind.SETTER));
}
static bool isStaticOrTopLevel(Element element) {
@@ -464,14 +456,14 @@ class Elements {
// `element.isTopLevel` is true.
if (Elements.isUnresolved(element)) return false;
if (element.isStatic || element.isTopLevel) return true;
- return !element.isAmbiguous
- && !element.isInstanceMember
- && !element.isPrefix
- && element.enclosingElement != null
- && (element.enclosingElement.kind == ElementKind.CLASS ||
- element.enclosingElement.kind == ElementKind.COMPILATION_UNIT ||
- element.enclosingElement.kind == ElementKind.LIBRARY ||
- element.enclosingElement.kind == ElementKind.PREFIX);
+ return !element.isAmbiguous &&
+ !element.isInstanceMember &&
+ !element.isPrefix &&
+ element.enclosingElement != null &&
+ (element.enclosingElement.kind == ElementKind.CLASS ||
+ element.enclosingElement.kind == ElementKind.COMPILATION_UNIT ||
+ element.enclosingElement.kind == ElementKind.LIBRARY ||
+ element.enclosingElement.kind == ElementKind.PREFIX);
}
static bool isInStaticContext(Element element) {
@@ -495,10 +487,10 @@ class Elements {
}
static bool isStaticOrTopLevelField(Element element) {
- return isStaticOrTopLevel(element)
- && (identical(element.kind, ElementKind.FIELD)
- || identical(element.kind, ElementKind.GETTER)
- || identical(element.kind, ElementKind.SETTER));
+ return isStaticOrTopLevel(element) &&
+ (identical(element.kind, ElementKind.FIELD) ||
+ identical(element.kind, ElementKind.GETTER) ||
+ identical(element.kind, ElementKind.SETTER));
}
static bool isStaticOrTopLevelFunction(Element element) {
@@ -506,9 +498,9 @@ class Elements {
}
static bool isInstanceMethod(Element element) {
- return !Elements.isUnresolved(element)
- && element.isInstanceMember
- && (identical(element.kind, ElementKind.FUNCTION));
+ return !Elements.isUnresolved(element) &&
+ element.isInstanceMember &&
+ (identical(element.kind, ElementKind.FUNCTION));
}
/// Also returns true for [ConstructorBodyElement]s and getters/setters.
@@ -526,8 +518,8 @@ class Elements {
Element element = elements[send];
if (element == null) return !isClosureSend(send, element);
return isInstanceMethod(element) ||
- isInstanceField(element) ||
- (send.isConditional && !element.isStatic);
+ isInstanceField(element) ||
+ (send.isConditional && !element.isStatic);
}
static bool isClosureSend(Send send, Element element) {
@@ -561,8 +553,8 @@ class Elements {
}
}
- static String constructorNameForDiagnostics(String className,
- String constructorName) {
+ static String constructorNameForDiagnostics(
+ String className, String constructorName) {
String classNameString = className;
String constructorNameString = constructorName;
return (constructorName == '')
@@ -644,8 +636,10 @@ class Elements {
static String constructOperatorName(String op, bool isUnary) {
String operatorName = constructOperatorNameOrNull(op, isUnary);
- if (operatorName == null) throw 'Unhandled operator: $op';
- else return operatorName;
+ if (operatorName == null)
+ throw 'Unhandled operator: $op';
+ else
+ return operatorName;
}
static String mapToUserOperatorOrNull(String op) {
@@ -706,49 +700,46 @@ class Elements {
return elements.toList()..sort(compareByPosition);
}
- static bool isFixedListConstructorCall(Element element,
- Send node,
- Compiler compiler) {
- return element == compiler.unnamedListConstructor
- && node.isCall
- && !node.arguments.isEmpty
- && node.arguments.tail.isEmpty;
+ static bool isFixedListConstructorCall(
+ Element element, Send node, Compiler compiler) {
+ return element == compiler.unnamedListConstructor &&
+ node.isCall &&
+ !node.arguments.isEmpty &&
+ node.arguments.tail.isEmpty;
}
- static bool isGrowableListConstructorCall(Element element,
- Send node,
- Compiler compiler) {
- return element == compiler.unnamedListConstructor
- && node.isCall
- && node.arguments.isEmpty;
+ static bool isGrowableListConstructorCall(
+ Element element, Send node, Compiler compiler) {
+ return element == compiler.unnamedListConstructor &&
+ node.isCall &&
+ node.arguments.isEmpty;
}
- static bool isFilledListConstructorCall(Element element,
- Send node,
- Compiler compiler) {
- return element == compiler.filledListConstructor
- && node.isCall
- && !node.arguments.isEmpty
- && !node.arguments.tail.isEmpty
- && node.arguments.tail.tail.isEmpty;
+ static bool isFilledListConstructorCall(
+ Element element, Send node, Compiler compiler) {
+ return element == compiler.filledListConstructor &&
+ node.isCall &&
+ !node.arguments.isEmpty &&
+ !node.arguments.tail.isEmpty &&
+ node.arguments.tail.tail.isEmpty;
}
- static bool isConstructorOfTypedArraySubclass(Element element,
- Compiler compiler) {
+ static bool isConstructorOfTypedArraySubclass(
+ Element element, Compiler compiler) {
if (compiler.typedDataLibrary == null) return false;
if (!element.isConstructor) return false;
ConstructorElement constructor = element.implementation;
constructor = constructor.effectiveTarget;
ClassElement cls = constructor.enclosingClass;
- return cls.library == compiler.typedDataLibrary
- && compiler.backend.isNative(cls)
- && compiler.world.isSubtypeOf(cls, compiler.typedDataClass)
- && compiler.world.isSubtypeOf(cls, compiler.coreClasses.listClass)
- && constructor.name == '';
+ return cls.library == compiler.typedDataLibrary &&
+ compiler.backend.isNative(cls) &&
+ compiler.world.isSubtypeOf(cls, compiler.typedDataClass) &&
+ compiler.world.isSubtypeOf(cls, compiler.coreClasses.listClass) &&
+ constructor.name == '';
}
- static bool switchStatementHasContinue(SwitchStatement node,
- TreeElements elements) {
+ static bool switchStatementHasContinue(
+ SwitchStatement node, TreeElements elements) {
for (SwitchCase switchCase in node.cases) {
for (Node labelOrCase in switchCase.labelsAndCases) {
Node label = labelOrCase.asLabel();
@@ -946,7 +937,6 @@ abstract class PrefixElement extends Element {
/// A type alias definition.
abstract class TypedefElement extends Element
implements AstElement, TypeDeclarationElement, FunctionTypedElement {
-
/// The type defined by this typedef with the type variables as its type
/// arguments.
///
@@ -997,8 +987,7 @@ abstract class MemberElement extends Element implements ExecutableElement {
/// A function, variable or parameter defined in an executable context.
abstract class LocalElement extends Element
- implements AstElement, TypedElement, Local {
-}
+ implements AstElement, TypedElement, Local {}
/// A top level, static or instance field, a formal parameter or local variable.
abstract class VariableElement extends ExecutableElement {
@@ -1033,12 +1022,10 @@ abstract class Local extends Entity {
/// The executable context is the [ExecutableElement] in which this variable
/// is defined.
abstract class LocalVariableElement extends VariableElement
- implements LocalElement {
-}
+ implements LocalElement {}
/// A top-level, static or instance field.
-abstract class FieldElement extends VariableElement implements MemberElement {
-}
+abstract class FieldElement extends VariableElement implements MemberElement {}
/// A parameter-like element of a function signature.
///
@@ -1086,8 +1073,7 @@ abstract class ParameterElement extends Element
/// A formal parameter on a function or constructor that introduces a local
/// variable in the scope of the function or constructor.
abstract class LocalParameterElement extends ParameterElement
- implements LocalVariableElement {
-}
+ implements LocalVariableElement {}
/// A formal parameter in a constructor that directly initializes a field.
///
@@ -1138,10 +1124,11 @@ abstract class FunctionSignature {
/// A top level, static or instance method, constructor, local function, or
/// closure (anonymous local function).
abstract class FunctionElement extends Element
- implements AstElement,
- TypedElement,
- FunctionTypedElement,
- ExecutableElement {
+ implements
+ AstElement,
+ TypedElement,
+ FunctionTypedElement,
+ ExecutableElement {
FunctionExpression get node;
FunctionElement get patch;
@@ -1212,14 +1199,11 @@ class AsyncMarker {
}
/// A top level, static or instance function.
-abstract class MethodElement extends FunctionElement
- implements MemberElement {
-}
+abstract class MethodElement extends FunctionElement implements MemberElement {}
/// A local function or closure (anonymous local function).
abstract class LocalFunctionElement extends FunctionElement
- implements LocalElement {
-}
+ implements LocalElement {}
/// A constructor.
abstract class ConstructorElement extends FunctionElement
@@ -1466,19 +1450,18 @@ abstract class ClassElement extends TypeDeclarationElement
Element lookupBackendMember(String memberName);
Element lookupSuperMember(String memberName);
- Element lookupSuperMemberInLibrary(String memberName,
- LibraryElement library);
+ Element lookupSuperMemberInLibrary(String memberName, LibraryElement library);
ConstructorElement lookupDefaultConstructor();
ConstructorElement lookupConstructor(String name);
void forEachMember(void f(ClassElement enclosingClass, Element member),
- {bool includeBackendMembers: false,
- bool includeSuperAndInjectedMembers: false});
+ {bool includeBackendMembers: false,
+ bool includeSuperAndInjectedMembers: false});
- void forEachInstanceField(void f(ClassElement enclosingClass,
- FieldElement field),
- {bool includeSuperAndInjectedMembers: false});
+ void forEachInstanceField(
+ void f(ClassElement enclosingClass, FieldElement field),
+ {bool includeSuperAndInjectedMembers: false});
/// Similar to [forEachInstanceField] but visits static fields.
void forEachStaticField(void f(ClassElement enclosingClass, Element field));
@@ -1723,4 +1706,3 @@ abstract class Member extends MemberSignature {
/// if any. Otherwise [implementation] points to the member itself.
Member get implementation;
}
-
« no previous file with comments | « pkg/compiler/lib/src/elements/common.dart ('k') | pkg/compiler/lib/src/elements/modelx.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698