[1/6] wsdapi: Add wsdapi_internal.h for shared internal function and macro definitions. [try 3]

Alexandre Julliard julliard at winehq.org
Tue Sep 12 09:19:57 CDT 2017


Owen Rudge <owen at owenrudge.net> writes:

> Hi Alexandre,
>
> Is there anything else that needs improving for these patches to be
> accepted?

I'm afraid that's just too much code at once. Part of it is the coding
style, where you use 30 lines for a function that should take 5, but
even taking that into account it's still too big to review carefully.
And from what little I've looked at, I'm afraid it's going to need a
very careful review.

-- 
Alexandre Julliard
julliard at winehq.org



More information about the wine-devel mailing list