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

Unified Diff: tools/ddbg.dart

Issue 10387232: Remove string concatenation with + from all Dart files in tools directory. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Address comments. Created 8 years, 7 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 | « no previous file | tools/test.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/ddbg.dart
diff --git a/tools/ddbg.dart b/tools/ddbg.dart
index 665b870e21c52b2035b901d5c3482006f84593e2..5ca8254d0558ba3824df848836e73a40e29e80e3 100644
--- a/tools/ddbg.dart
+++ b/tools/ddbg.dart
@@ -67,30 +67,30 @@ void processCommand(String cmdLine) {
if (args.length == 0) {
return;
}
- var cmd = args[0];
- if (cmd == "r") {
+ var command = args[0];
+ if (command == "r") {
var cmd = { "id": seqNum, "command": "resume" };
sendCmd(cmd).then((result) => handleGenericResponse(result));
stackTrace = curFrame = null;
- } else if (cmd == "s") {
+ } else if (command == "s") {
var cmd = { "id": seqNum, "command": "stepOver" };
sendCmd(cmd).then((result) => handleGenericResponse(result));
stackTrace = curFrame = null;
- } else if (cmd == "si") {
+ } else if (command == "si") {
var cmd = { "id": seqNum, "command": "stepInto" };
sendCmd(cmd).then((result) => handleGenericResponse(result));
stackTrace = curFrame = null;
- } else if (cmd == "so") {
+ } else if (command == "so") {
var cmd = { "id": seqNum, "command": "stepOut" };
sendCmd(cmd).then((result) => handleGenericResponse(result));
stackTrace = curFrame = null;
- } else if (cmd == "bt") {
+ } else if (command == "bt") {
var cmd = { "id": seqNum, "command": "getStackTrace" };
sendCmd(cmd).then((result) => handleStackTraceResponse(result));
- } else if (cmd == "ll") {
+ } else if (command == "ll") {
var cmd = { "id": seqNum, "command": "getLibraryURLs" };
sendCmd(cmd).then((result) => handleGetLibraryResponse(result));
- } else if (cmd == "sbp" && args.length >= 2) {
+ } else if (command == "sbp" && args.length >= 2) {
var url, line;
if (args.length == 2) {
url = stackTrace[0]["location"]["url"];
@@ -103,25 +103,25 @@ void processCommand(String cmdLine) {
"command": "setBreakpoint",
"params": { "url": url, "line": line }};
sendCmd(cmd).then((result) => handleSetBpResponse(result));
- } else if (cmd == "ls" && args.length == 2) {
+ } else if (command == "ls" && args.length == 2) {
var cmd = { "id": seqNum,
"command": "getScriptURLs",
"params": { "library": args[1] }};
sendCmd(cmd).then((result) => handleGetScriptsResponse(result));
- } else if (cmd == "po" && args.length == 2) {
+ } else if (command == "po" && args.length == 2) {
var cmd = { "id": seqNum, "command": "getObjectProperties",
"params": {"objectId": Math.parseInt(args[1]) }};
sendCmd(cmd).then((result) => handleGetObjPropsResponse(result));
- } else if (cmd == "pc" && args.length == 2) {
+ } else if (command == "pc" && args.length == 2) {
var cmd = { "id": seqNum, "command": "getClassProperties",
"params": {"classId": Math.parseInt(args[1]) }};
sendCmd(cmd).then((result) => handleGetClassPropsResponse(result));
- } else if (cmd == "q") {
+ } else if (command == "q") {
quitShell();
- } else if (cmd == "h") {
+ } else if (command == "h") {
printHelp();
} else {
- print("command '$cmd' not understood, try h for help");
+ print("command '$command' not understood, try h for help");
}
}
« no previous file with comments | « no previous file | tools/test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698