Revision: 12433
Author:   [email protected]
Date:     Mon Sep  3 08:50:24 2012
Log:      Fix more windows build warnings.

[email protected]
BUG=

Review URL: https://chromiumcodereview.appspot.com/10917065
http://code.google.com/p/v8/source/detail?r=12433

Modified:
 /branches/bleeding_edge/src/api.cc
 /branches/bleeding_edge/test/cctest/test-api.cc

=======================================
--- /branches/bleeding_edge/src/api.cc  Mon Sep  3 08:40:00 2012
+++ /branches/bleeding_edge/src/api.cc  Mon Sep  3 08:50:24 2012
@@ -4961,7 +4961,8 @@
// since it's not. Instead, we convert it to an internal string and dispose
     // the external resource.
     result = isolate->factory()->NewStringFromLatin1(
-        i::Vector<const char>(resource->data(), resource->length()),
+        i::Vector<const char>(resource->data(),
+                              static_cast<int>(resource->length())),
         i::NOT_TENURED,
         i::String::NOT_ASCII);
     resource->Dispose();
=======================================
--- /branches/bleeding_edge/test/cctest/test-api.cc     Mon Sep  3 08:06:36 2012
+++ /branches/bleeding_edge/test/cctest/test-api.cc     Mon Sep  3 08:50:24 2012
@@ -717,10 +717,10 @@
 static void TestNewLatin1String(int encoding1, int encoding2) {
   const char* chars1 = "ASCII 123";
   const char* chars1js = "'ASCII 123'";
-  int str1_len = strlen(chars1);
+  int str1_len = static_cast<int>(strlen(chars1));
   const char* chars2 = "Non-ASCII \xAB\xCD\xEF";
   const char* chars2js = "'Non-ASCII \\u00ab\\u00cd\\u00ef'";
-  int str2_len = strlen(chars2);
+  int str2_len = static_cast<int>(strlen(chars2));

   Local<String> str1 = String::New(chars1, str1_len, encoding1);
   Local<String> str2 = String::New(chars2, str2_len, encoding2);
@@ -826,17 +826,21 @@
   CHECK(v8::Utils::OpenHandle(*sub_string)->IsSlicedString());

   char buffer[64];
-  CHECK_EQ(strlen(latin1_ascii), latin1_ascii_string->WriteLatin1(buffer));
+  CHECK_EQ(static_cast<int>(strlen(latin1_ascii)),
+           latin1_ascii_string->WriteLatin1(buffer));
   CHECK_EQ(0, strcmp(latin1_ascii, buffer));
-  CHECK_EQ(strlen(latin1), latin1_string->WriteLatin1(buffer));
+  CHECK_EQ(static_cast<int>(strlen(latin1)),
+           latin1_string->WriteLatin1(buffer));
   CHECK_EQ(0, strcmp(latin1, buffer));
-  CHECK_EQ(strlen(concat), concat_string->WriteLatin1(buffer));
+  CHECK_EQ(static_cast<int>(strlen(concat)),
+           concat_string->WriteLatin1(buffer));
   CHECK_EQ(0, strcmp(concat, buffer));
-  CHECK_EQ(strlen(sub), sub_string->WriteLatin1(buffer));
+  CHECK_EQ(static_cast<int>(strlen(sub)),
+           sub_string->WriteLatin1(buffer));
   CHECK_EQ(0, strcmp(sub, buffer));

   memset(buffer, 0x1, sizeof(buffer));
-  CHECK_EQ(strlen(latin1),
+  CHECK_EQ(static_cast<int>(strlen(latin1)),
            latin1_string->WriteLatin1(buffer,
                                       0,
                                       String::kUndefinedLength,
@@ -846,7 +850,7 @@
   buffer[strlen(latin1)] = '\0';
   CHECK_EQ(0, strcmp(latin1, buffer));

-  CHECK_EQ(strlen(latin1) - 2,
+  CHECK_EQ(static_cast<int>(strlen(latin1)) - 2,
            latin1_string->WriteLatin1(buffer, 2));
   CHECK_EQ(0, strncmp(latin1 + 2, buffer, strlen(latin1)));
 }

--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to