[PATCH] kernel32: fix command line argument escaping in build_command_line

Alexandre Julliard julliard at winehq.org
Wed Feb 8 02:25:55 CST 2017


Jacob Lifshay <programmerjake at gmail.com> writes:

> The old loop seemed confusing to me and it overestimated the size of
> the memory to allocate by a large factor. The old loop would need to
> have about half of it rewritten to fix the size overestimation and a
> little more for fixing the original bug. If you think it's very
> important to keep as much of the old code as possible, I can rewrite
> the patch that way, but I don't think that's necessary.

Please try to fix separate issues as separate patches instead of
rewriting everything. The overestimation is very unlikely to be a
problem; in fact I'd say that something simple like doubling the
incoming size would be better than a complicated calculation.

-- 
Alexandre Julliard
julliard at winehq.org



More information about the wine-devel mailing list