[PATCH 1/3] cabinet/tests: Alter string declaration to include null terminator.

Isira Seneviratne isirasen96 at gmail.com
Sun Aug 18 17:51:07 CDT 2019


On Mon, Aug 19, 2019, 1:42 AM Alex Henrie <alexhenrie24 at gmail.com> wrote:

> + * Copyright (C) 2019 Isira Seneviratne
>
> -    static const char expected[12] = "Hello World!";
> +    static const char expected[12] = {
> 'H','e','l','l','o','W','o','r','l','d','!','\0' };
>
> All you had to do was remove the number 12 and let the compiler
> determine the size of the array automatically. And you really
> shouldn't add your name at the top of the file unless you wrote a
> substantial chunk of the file.
>
> -Alex
>

Okay, thanks.

By the way, why would the tests fail?

>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.winehq.org/pipermail/wine-devel/attachments/20190819/e4b2e2c9/attachment.html>


More information about the wine-devel mailing list