Re: [PATCH 0/5] MR8879: opcservices/tests: Use message context in some tests.

[Vibhav Pant (＠vibhavp)]

Vibhav Pant (@vibhavp) commented about dlls/opcservices/compress.c:
>      return E_OUTOFMEMORY;
>  WideCharToMultiByte(CP_ACP, 0, path, -1, name, len, NULL, NULL);
>
> - /* Local header */
> - if (!(file = calloc(1, sizeof(*file) + len)))

Would it be a good idea to add a `char name[0]` at the end of `struct zip_file`? That might make the name-related code a little clearer (`calloc(1, offsetof(struct zip_file, name[len]), compress_write(archive,file->name, file->name_length)`).