Re: [PATCH] shlwapi: Changed UrlEscapeW error paths and add tests

[Fabian Maurer]

On Tuesday, December 13, 2016 11:13:51 PM CET Jacek Caban wrote:
> Please send wininet changes as separated patch. They don't depend on
> shlwapi, so may be applied separately, before shlwapi patch.

Well they do kind of depend on the shlwapi changes. HTTP_HandleRedirect relies 
on wine behaviour that doesn't work on windows. With only the shlwapi changes, 
HTTP_HandleRedirect would break. Should I still split them up?

On Tuesday, December 13, 2016 11:13:51 PM CET Jacek Caban wrote:
> You set and test size although the actual call doesn't use it. Please
> remove lines with size. Same in a few more cases.

Oh yes, I missed that. Thanks for pointing it out.

Fabian