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

Unified Diff: sdk/lib/mirrors/mirrors.dart

Issue 392983006: Specifiy MethodMirror.isStatic is terms of whether 'this' is permitted. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 6 years, 5 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 | « runtime/lib/mirrors.cc ('k') | tests/lib/mirrors/local_function_is_static_test.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sdk/lib/mirrors/mirrors.dart
diff --git a/sdk/lib/mirrors/mirrors.dart b/sdk/lib/mirrors/mirrors.dart
index 6d7717bd9eea841ff240f5e3f39fce1b22cba1cb..fb90271d96de37057c83c60b6a095ee844d5baae 100644
--- a/sdk/lib/mirrors/mirrors.dart
+++ b/sdk/lib/mirrors/mirrors.dart
@@ -900,10 +900,11 @@ abstract class MethodMirror implements DeclarationMirror {
List<ParameterMirror> get parameters;
/**
- * Is the reflectee static?
+ * A method is that is top-level or a static member of a class is considered
+ * static. All other methods are considered non-static.
*
- * For the purposes of the mirrors library, a top-level function is
- * considered static.
+ * Note that generative constructors are non-static, whereas factory
+ * constructors are static.
gbracha 2014/07/16 21:01:33 "A function" rather than a "A method" would be bet
*/
bool get isStatic;
« no previous file with comments | « runtime/lib/mirrors.cc ('k') | tests/lib/mirrors/local_function_is_static_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698