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

Unified Diff: runtime/bin/platform_win.cc

Issue 317773002: Fix Win64 build of Dart VM. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: add fix for http://dartbug.com/19213 Created 6 years, 6 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
Index: runtime/bin/platform_win.cc
diff --git a/runtime/bin/platform_win.cc b/runtime/bin/platform_win.cc
index 85b6694d47dba42a51d60eb42c083047b95ec09f..8a155d54ffd3e13533fc4c33ce0395fc40304cd6 100644
--- a/runtime/bin/platform_win.cc
+++ b/runtime/bin/platform_win.cc
@@ -59,7 +59,7 @@ char** Platform::Environment(intptr_t* count) {
}
-void Platform::FreeEnvironment(char** env, int count) {
+void Platform::FreeEnvironment(char** env, intptr_t count) {
for (int i = 0; i < count; i++) free(env[i]);
siva 2014/06/05 17:21:18 should this also be for (intptr_t i = 0, ...)
Ivan Posva 2014/06/05 17:42:11 "intptr_t i" in that case too? And {} please...
Vyacheslav Egorov (Google) 2014/06/05 18:05:55 Done.
Vyacheslav Egorov (Google) 2014/06/05 18:05:55 Done.
delete[] env;
}

Powered by Google App Engine
This is Rietveld 408576698