Piotr Caban : msvcp90/tests: Remove unneeded casts.

Alexandre Julliard julliard at winehq.org
Thu Jun 23 11:15:24 CDT 2016


Module: wine
Branch: master
Commit: c6530fee82352573e12f3dc1e3bf5edbef10807f
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=c6530fee82352573e12f3dc1e3bf5edbef10807f

Author: Piotr Caban <piotr at codeweavers.com>
Date:   Thu Jun 23 10:24:20 2016 +0200

msvcp90/tests: Remove unneeded casts.

Signed-off-by: Piotr Caban <piotr at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 dlls/msvcp90/tests/ios.c    |  8 ++++----
 dlls/msvcp90/tests/string.c | 28 ++++++++++++++--------------
 2 files changed, 18 insertions(+), 18 deletions(-)

diff --git a/dlls/msvcp90/tests/ios.c b/dlls/msvcp90/tests/ios.c
index 0c57ca1..91c546e 100644
--- a/dlls/msvcp90/tests/ios.c
+++ b/dlls/msvcp90/tests/ios.c
@@ -1833,7 +1833,7 @@ static void test_istream_tellg(void)
 
         if (tests[i].seekoff != -1) /* to test without seek */
             call_func2_ptr_fpos(p_basic_istream_char_seekg_fpos, &ss.base.base1, spos);
-        rpos = (fpos_int *)call_func2(p_basic_istream_char_tellg, &ss.base.base1, &tpos);
+        rpos = call_func2(p_basic_istream_char_tellg, &ss.base.base1, &tpos);
 
         ok(tests[i].telloff_ss == tpos.off, "wrong offset, expected = %ld found = %ld\n", tests[i].telloff_ss, tpos.off);
         if (tests[i].telloff_ss != -1 && spos.off != -1) /* check if tell == seek but only if not hit EOF */
@@ -1860,7 +1860,7 @@ static void test_istream_tellg(void)
 
         if (tests[i].seekoff != -1) /* to test without seek */
             call_func2_ptr_fpos(p_basic_istream_wchar_seekg_fpos, &wss.base.base1, spos);
-        rpos = (fpos_int *)call_func2(p_basic_istream_wchar_tellg, &wss.base.base1, &tpos);
+        rpos = call_func2(p_basic_istream_wchar_tellg, &wss.base.base1, &tpos);
 
         ok(tests[i].telloff_ss == tpos.off, "wrong offset, expected = %ld found = %ld\n", tests[i].telloff_ss, tpos.off);
         if (tests[i].telloff_ss != -1 && spos.off != -1) /* check if tell == seek but only if not hit EOF */
@@ -1890,7 +1890,7 @@ static void test_istream_tellg(void)
 
         if (tests[i].seekoff != -1) /* to test without seek */
             call_func2_ptr_fpos(p_basic_istream_char_seekg_fpos, &fs.base.base1, spos);
-        rpos = (fpos_int *)call_func2(p_basic_istream_char_tellg, &fs.base.base1, &tpos);
+        rpos = call_func2(p_basic_istream_char_tellg, &fs.base.base1, &tpos);
 
         ok(tests[i].tellpos == tpos.pos, "wrong filepos, expected = %s found = %s\n",
             debugstr_longlong(tests[i].tellpos), debugstr_longlong(tpos.pos));
@@ -1913,7 +1913,7 @@ static void test_istream_tellg(void)
 
         if (tests[i].seekoff != -1) /* to test without seek */
             call_func2_ptr_fpos(p_basic_istream_wchar_seekg_fpos, &wfs.base.base1, spos);
-        rpos = (fpos_int *)call_func2(p_basic_istream_wchar_tellg, &wfs.base.base1, &tpos);
+        rpos = call_func2(p_basic_istream_wchar_tellg, &wfs.base.base1, &tpos);
 
         ok(tests[i].tellpos == tpos.pos, "wrong filepos, expected = %s found = %s\n",
             debugstr_longlong(tests[i].tellpos), debugstr_longlong(tpos.pos));
diff --git a/dlls/msvcp90/tests/string.c b/dlls/msvcp90/tests/string.c
index 8aea30a..9d46df0 100644
--- a/dlls/msvcp90/tests/string.c
+++ b/dlls/msvcp90/tests/string.c
@@ -409,9 +409,9 @@ static void test_basic_string_char_swap(void) {
     strcpy(atmp1, "qwerty");
     call_func2(p_basic_string_char_ctor_cstr, &str1, atmp1);
     call_func2(p_basic_string_char_swap, &str1, &str1);
-    ok(strcmp(atmp1, (const char *) call_func1(p_basic_string_char_cstr, &str1)) == 0, "Invalid value of str1\n");
+    ok(strcmp(atmp1, call_func1(p_basic_string_char_cstr, &str1)) == 0, "Invalid value of str1\n");
     call_func2(p_basic_string_char_swap, &str1, &str1);
-    ok(strcmp(atmp1, (const char *) call_func1(p_basic_string_char_cstr, &str1)) == 0, "Invalid value of str1\n");
+    ok(strcmp(atmp1, call_func1(p_basic_string_char_cstr, &str1)) == 0, "Invalid value of str1\n");
     call_func1(p_basic_string_char_dtor, &str1);
 
     /* str1 allocated, str2 local */
@@ -420,11 +420,11 @@ static void test_basic_string_char_swap(void) {
     call_func2(p_basic_string_char_ctor_cstr, &str1, atmp1);
     call_func2(p_basic_string_char_ctor_cstr, &str2, atmp2);
     call_func2(p_basic_string_char_swap, &str1, &str2);
-    ok(strcmp(atmp2, (const char *) call_func1(p_basic_string_char_cstr, &str1)) == 0, "Invalid value of str1\n");
-    ok(strcmp(atmp1, (const char *) call_func1(p_basic_string_char_cstr, &str2)) == 0, "Invalid value of str2\n");
+    ok(strcmp(atmp2, call_func1(p_basic_string_char_cstr, &str1)) == 0, "Invalid value of str1\n");
+    ok(strcmp(atmp1, call_func1(p_basic_string_char_cstr, &str2)) == 0, "Invalid value of str2\n");
     call_func2(p_basic_string_char_swap, &str1, &str2);
-    ok(strcmp(atmp1, (const char *) call_func1(p_basic_string_char_cstr, &str1)) == 0, "Invalid value of str1\n");
-    ok(strcmp(atmp2, (const char *) call_func1(p_basic_string_char_cstr, &str2)) == 0, "Invalid value of str2\n");
+    ok(strcmp(atmp1, call_func1(p_basic_string_char_cstr, &str1)) == 0, "Invalid value of str1\n");
+    ok(strcmp(atmp2, call_func1(p_basic_string_char_cstr, &str2)) == 0, "Invalid value of str2\n");
     call_func1(p_basic_string_char_dtor, &str1);
     call_func1(p_basic_string_char_dtor, &str2);
 }
@@ -607,8 +607,8 @@ static void test_basic_string_char_replace(void) {
 
         ret = call_func4(p_basic_string_char_replace_cstr, &str, tests[i].off, tests[i].len, tests[i].replace);
         ok(ret == &str, "str = %p ret = %p\n", ret, &str);
-        ok(strcmp(tests[i].ret, (const char *) call_func1(p_basic_string_char_cstr, ret)) == 0, "str = %s ret = %s\n",
-                  tests[i].ret, (const char *) call_func1(p_basic_string_char_cstr, ret));
+        ok(strcmp(tests[i].ret, call_func1(p_basic_string_char_cstr, ret)) == 0, "str = %s ret = %s\n",
+                  tests[i].ret, (const char*)call_func1(p_basic_string_char_cstr, ret));
 
         call_func1(p_basic_string_char_dtor, &str);
     }
@@ -694,9 +694,9 @@ static void test_basic_string_wchar_swap(void) {
     mbstowcs(wtmp1, "qwerty", 32);
     call_func2(p_basic_string_wchar_ctor_cstr, &str1, wtmp1);
     call_func2(p_basic_string_wchar_swap, &str1, &str1);
-    ok(wcscmp(wtmp1, (const wchar_t *) call_func1(p_basic_string_wchar_cstr, &str1)) == 0, "Invalid value of str1\n");
+    ok(wcscmp(wtmp1, call_func1(p_basic_string_wchar_cstr, &str1)) == 0, "Invalid value of str1\n");
     call_func2(p_basic_string_wchar_swap, &str1, &str1);
-    ok(wcscmp(wtmp1, (const wchar_t *) call_func1(p_basic_string_wchar_cstr, &str1)) == 0, "Invalid value of str1\n");
+    ok(wcscmp(wtmp1, call_func1(p_basic_string_wchar_cstr, &str1)) == 0, "Invalid value of str1\n");
     call_func1(p_basic_string_wchar_dtor, &str1);
 
     /* str1 allocated, str2 local */
@@ -705,11 +705,11 @@ static void test_basic_string_wchar_swap(void) {
     call_func2(p_basic_string_wchar_ctor_cstr, &str1, wtmp1);
     call_func2(p_basic_string_wchar_ctor_cstr, &str2, wtmp2);
     call_func2(p_basic_string_wchar_swap, &str1, &str2);
-    ok(wcscmp(wtmp2, (const wchar_t *) call_func1(p_basic_string_wchar_cstr, &str1)) == 0, "Invalid value of str1\n");
-    ok(wcscmp(wtmp1, (const wchar_t *) call_func1(p_basic_string_wchar_cstr, &str2)) == 0, "Invalid value of str2\n");
+    ok(wcscmp(wtmp2, call_func1(p_basic_string_wchar_cstr, &str1)) == 0, "Invalid value of str1\n");
+    ok(wcscmp(wtmp1, call_func1(p_basic_string_wchar_cstr, &str2)) == 0, "Invalid value of str2\n");
     call_func2(p_basic_string_wchar_swap, &str1, &str2);
-    ok(wcscmp(wtmp1, (const wchar_t *) call_func1(p_basic_string_wchar_cstr, &str1)) == 0, "Invalid value of str1\n");
-    ok(wcscmp(wtmp2, (const wchar_t *) call_func1(p_basic_string_wchar_cstr, &str2)) == 0, "Invalid value of str2\n");
+    ok(wcscmp(wtmp1, call_func1(p_basic_string_wchar_cstr, &str1)) == 0, "Invalid value of str1\n");
+    ok(wcscmp(wtmp2, call_func1(p_basic_string_wchar_cstr, &str2)) == 0, "Invalid value of str2\n");
     call_func1(p_basic_string_wchar_dtor, &str1);
     call_func1(p_basic_string_wchar_dtor, &str2);
 }




More information about the wine-cvs mailing list