Commit 3dd1a8b4 authored by Francois Gouget's avatar Francois Gouget Committed by Alexandre Julliard

oleaut32/tests: Replace some '#if 0's with 'if (0)'s.

parent ac25a3f9
...@@ -75,7 +75,6 @@ static const unsigned char jpgimage[285] = { ...@@ -75,7 +75,6 @@ static const unsigned char jpgimage[285] = {
0x00,0x02,0x11,0x03,0x11,0x00,0x3f,0x00,0xb2,0xc0,0x07,0xff,0xd9 0x00,0x02,0x11,0x03,0x11,0x00,0x3f,0x00,0xb2,0xc0,0x07,0xff,0xd9
}; };
#if 0 /* no png support yet */
/* 1x1 pixel png */ /* 1x1 pixel png */
static const unsigned char pngimage[285] = { static const unsigned char pngimage[285] = {
0x89,0x50,0x4e,0x47,0x0d,0x0a,0x1a,0x0a,0x00,0x00,0x00,0x0d,0x49,0x48,0x44,0x52, 0x89,0x50,0x4e,0x47,0x0d,0x0a,0x1a,0x0a,0x00,0x00,0x00,0x0d,0x49,0x48,0x44,0x52,
...@@ -86,7 +85,6 @@ static const unsigned char pngimage[285] = { ...@@ -86,7 +85,6 @@ static const unsigned char pngimage[285] = {
0x54,0x08,0xd7,0x63,0xf8,0xff,0xff,0x3f,0x00,0x05,0xfe,0x02,0xfe,0xdc,0xcc,0x59, 0x54,0x08,0xd7,0x63,0xf8,0xff,0xff,0x3f,0x00,0x05,0xfe,0x02,0xfe,0xdc,0xcc,0x59,
0xe7,0x00,0x00,0x00,0x00,0x49,0x45,0x4e,0x44,0xae,0x42,0x60,0x82 0xe7,0x00,0x00,0x00,0x00,0x49,0x45,0x4e,0x44,0xae,0x42,0x60,0x82
}; };
#endif
/* 1x1 pixel bmp */ /* 1x1 pixel bmp */
static const unsigned char bmpimage[66] = { static const unsigned char bmpimage[66] = {
...@@ -410,9 +408,8 @@ START_TEST(olepicture) ...@@ -410,9 +408,8 @@ START_TEST(olepicture)
test_pic(jpgimage, sizeof(jpgimage)); test_pic(jpgimage, sizeof(jpgimage));
test_pic(bmpimage, sizeof(bmpimage)); test_pic(bmpimage, sizeof(bmpimage));
test_pic(gif4pixel, sizeof(gif4pixel)); test_pic(gif4pixel, sizeof(gif4pixel));
/* No PNG support yet here or in older Windows... /* FIXME: No PNG support yet in Wine or in older Windows... */
test_pic(pngimage, sizeof(pngimage)); if (0) test_pic(pngimage, sizeof(pngimage));
*/
test_empty_image(); test_empty_image();
test_empty_image_2(); test_empty_image_2();
......
...@@ -649,11 +649,12 @@ static void test_SafeArrayAllocDestroyDescriptor(void) ...@@ -649,11 +649,12 @@ static void test_SafeArrayAllocDestroyDescriptor(void)
ok(IS_ANCIENT || hres == E_INVALIDARG, ok(IS_ANCIENT || hres == E_INVALIDARG,
"65536 dimensions gave hres 0x%x\n", hres); "65536 dimensions gave hres 0x%x\n", hres);
#if 0 if (0)
{
/* Crashes on 95: XP & Wine return E_POINTER */ /* Crashes on 95: XP & Wine return E_POINTER */
hres=SafeArrayAllocDescriptor(1, NULL); hres=SafeArrayAllocDescriptor(1, NULL);
ok(hres == E_POINTER,"NULL parm gave hres 0x%x\n", hres); ok(hres == E_POINTER,"NULL parm gave hres 0x%x\n", hres);
#endif }
/* Test up to the dimension boundary case */ /* Test up to the dimension boundary case */
for (i = 5; i <= 65535; i += 30) for (i = 5; i <= 65535; i += 30)
...@@ -985,13 +986,14 @@ static void test_SafeArrayGetPutElement(void) ...@@ -985,13 +986,14 @@ static void test_SafeArrayGetPutElement(void)
hres = SafeArrayGetElement(sa, NULL, &value); hres = SafeArrayGetElement(sa, NULL, &value);
ok(hres == E_INVALIDARG, "Get NULL indices hres 0x%x\n", hres); ok(hres == E_INVALIDARG, "Get NULL indices hres 0x%x\n", hres);
#if 0 if (0)
{
/* This is retarded. Windows checks every case of invalid parameters /* This is retarded. Windows checks every case of invalid parameters
* except the following, which crashes. We ERR this in Wine. * except the following, which crashes. We ERR this in Wine.
*/ */
hres = SafeArrayPutElement(sa, indices, NULL); hres = SafeArrayPutElement(sa, indices, NULL);
ok(hres == E_INVALIDARG, "Put NULL value hres 0x%x\n", hres); ok(hres == E_INVALIDARG, "Put NULL value hres 0x%x\n", hres);
#endif }
hres = SafeArrayGetElement(sa, indices, NULL); hres = SafeArrayGetElement(sa, indices, NULL);
ok(hres == E_INVALIDARG, "Get NULL value hres 0x%x\n", hres); ok(hres == E_INVALIDARG, "Get NULL value hres 0x%x\n", hres);
......
...@@ -316,10 +316,8 @@ static void test_VariantClear(void) ...@@ -316,10 +316,8 @@ static void test_VariantClear(void)
LONG i4; LONG i4;
IUnknown *punk; IUnknown *punk;
#if 0
/* Crashes: Native does not test input for NULL, so neither does Wine */ /* Crashes: Native does not test input for NULL, so neither does Wine */
hres = VariantClear(NULL); if (0) hres = VariantClear(NULL);
#endif
/* Only the type field is set, to VT_EMPTY */ /* Only the type field is set, to VT_EMPTY */
V_VT(&v) = VT_UI4; V_VT(&v) = VT_UI4;
......
...@@ -2558,10 +2558,12 @@ static void test_VarUI8FromDec(void) ...@@ -2558,10 +2558,12 @@ static void test_VarUI8FromDec(void)
CONVERT_BADDEC(VarUI8FromDec); CONVERT_BADDEC(VarUI8FromDec);
#if 0
/* This returns 1 under native; Wine fixes this bug and returns overflow */ /* This returns 1 under native; Wine fixes this bug and returns overflow */
CONVERT_DEC(VarUI8FromDec,0,0x80,0,1); if (0)
#endif {
CONVERT_DEC(VarUI8FromDec,0,0x80,0,1);
}
CONVERT_DEC(VarUI8FromDec,0,0,0,0); EXPECTI8(0); CONVERT_DEC(VarUI8FromDec,0,0,0,0); EXPECTI8(0);
CONVERT_DEC(VarUI8FromDec,0,0,0,1); EXPECTI8(1); CONVERT_DEC(VarUI8FromDec,0,0,0,1); EXPECTI8(1);
CONVERT_DEC(VarUI8FromDec,0,0,0,255); EXPECTI8(255); CONVERT_DEC(VarUI8FromDec,0,0,0,255); EXPECTI8(255);
...@@ -5012,12 +5014,13 @@ static void test_SysAllocStringLen(void) ...@@ -5012,12 +5014,13 @@ static void test_SysAllocStringLen(void)
BSTR str; BSTR str;
/* Very early native dlls do not limit the size of strings, so skip this test */ /* Very early native dlls do not limit the size of strings, so skip this test */
#if 0 if (0)
{
str = SysAllocStringLen(szTest, 0x80000000); str = SysAllocStringLen(szTest, 0x80000000);
todo_wine { todo_wine {
ok (str == NULL, "Expected NULL, got %p\n", str); ok (str == NULL, "Expected NULL, got %p\n", str);
} }
#endif }
str = SysAllocStringLen(NULL, 0); str = SysAllocStringLen(NULL, 0);
ok (str != NULL, "Expected non-NULL\n"); ok (str != NULL, "Expected non-NULL\n");
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment