Re: [PATCH v11 0/2] MR1268: wininet: Partially implement InternetSetFilePointer

[Jacek Caban (＠jacek)]

Jacek Caban (@jacek) commented about dlls/wininet/tests/http.c:
>  CHECK_NOTIFIED2(INTERNET_STATUS_HANDLE_CLOSING, handle_cnt);
>
> }
>
> +static void InternetSetFilePointer_test(const char *host, const char *path)
> +{
> +#ifndef BUFFER_SIZE
> +#define BUFFER_SIZE 8192
> +#endif

#ifndef part is redundant. I think it would be cleaner just to hardcode it in buffer declaration and use sizeof() later.