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

Unified Diff: frog/frog_options.dart

Issue 9129023: adds array bounds checking (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: merged Created 8 years, 11 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 | « frog/corejs.dart ('k') | frog/gen.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: frog/frog_options.dart
diff --git a/frog/frog_options.dart b/frog/frog_options.dart
index 67e99a00f0cebf7a9437f5fa67622a5d2f1005f3..5ce8ed5e9e07e71e08da65994c3337d31d7de169 100644
--- a/frog/frog_options.dart
+++ b/frog/frog_options.dart
@@ -39,6 +39,10 @@ class FrogOptions {
bool compileOnly = false;
bool inferTypes = false;
+ // Specifies non-compliant behavior where array bounds checks are
+ // not implemented in generated code.
+ bool disableBoundsChecks = false;
+
// Message support
bool throwOnErrors = false;
bool throwOnWarnings = false;
@@ -150,6 +154,15 @@ class FrogOptions {
inferTypes = true;
break;
+ case '--checked':
+ enableTypeChecks = true;
+ enableAsserts = true;
+ break;
+
+ case '--unchecked':
+ disableBoundsChecks = true;
+ break;
+
default:
if (arg.endsWith('.dart')) {
dartScript = arg;
« no previous file with comments | « frog/corejs.dart ('k') | frog/gen.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698