shell32: Initialize all of wszDstPath in ISFHelper_fnCopyItems. ISFHelper_fnCopyItems.

Gerald Pfeifer gerald at pfeifer.com
Sun May 22 14:53:42 CDT 2016


On Wed, 18 May 2016, Alexandre Julliard wrote:
>> wszDstPath is an array of WCHAR, which are wider than one byte,
>> so this should have been (MAXPATH+1)*sizeof(WCHAR) -- or the more
>> robust approach I here propose.
> Actually there's no reason to initialize the entire buffer.

And MAXPATH+1 is correct?  Or is that arbitrary as well?

Initialized an array of WCHAR to the size the array would have
were it an array of char feels quite unintuitive, so at least a
comment would be good.

Gerald



More information about the wine-devel mailing list