29 Aug
2015
29 Aug
'15
6:19 p.m.
On Sun, Aug 30, 2015 at 2:13 AM, Matt Durgavich mattdurgavich@gmail.com wrote:
All suggestions rolled in. Removing todos in tests will be a separate patch
You can't do that, because the changes you are making will break the tests (tests will fail since problems were fixed and todo_wine need to be removed). Just merge the todo removal and IMO the patch is fine. Remember to run make test inside the ws2_32/tests folder to check if it runs fine.
It amazes me that such a simple solution could be found for this bug, thanks for working on it and thanks Henri for the tip.