Re: msvcrt: Fixed (_w)fopen_s with test. (try 2)

[Dmitry Timoshkov]

Eryk Wieliczko <ewdevel(a)gmail.com> wrote:
> +    if (!p_fopen_s)
> +    {
> +        win_skip("Skipping fopen_s test\n");
> +        return;
> +    }
> +    else
> +    {
> +        /* testing fopen_s */
> +        ret = p_fopen_s(&file, name, "w");
> +        ok(ret == 0, "fopen_s failed with %d\n", ret);
> +        ok(file != 0, "fopen_s failed to return value\n");
> +        fwrite(name, sizeof(name), 1, file);
> +
> +        ret = fclose(file);
> +        ok(ret != EOF, "File failed to close\n");
> +
> +        file = fopen(name, "r");
> +        ok(file != 0, "fopen failed\n");
> +        len = fread(buff, 1, sizeof(name), file);
> +        ok(len == sizeof(name), "File length supposed to be %d, not %d\n", sizeof(name), len);
> +        buff[sizeof(name)] = '\0';
> +        ok(strcmp(name, buff) == 0, "File content mismatch! Got %s, expected %s\n", buff, name);
> +
> +        ret = fclose(file);
> +        ok(ret != EOF, "File failed to close\n");
> +
> +        ok(_unlink(name) == 0, "Couldn't unlink file named '%s'\n", name);
> +    }
> +}

The whole 'else { }' block is not necessary, same for the test below.

-- 
Dmitry.