kernel32: Implement ReplaceFileA/ReplaceFileW

Felix Nawothnig flexo at holycrap.org
Fri Mar 2 12:52:11 CST 2007


Erich Hoover wrote:
>> I see your point. However, since the function you are implementing is in
>> kernel32 anyway you could abstract it away and make both functions
>> (CopyFile and ReplaceFile) call some internal function. That way you
>> would get rid of the locking completly which is argueably somewhat ugly.
> On closer inspection, CreateFile actually seems to take care of this 
> with the "security attributes" and "template" parameters.  It does not 
> look like the security attributes are implemented yet (or the call to 
> get the security attributes for a file) but since CreateFile already 
> handles this it seems unnecessary to add a new break-out function.  You 
> will likely find the attached more to your liking.

Doesn't matter what I like, in the end it's up to Alexandre.

But yes, looks much better to me (although I'd still say it makes sense 
to abstract away the actually copying in a subroutine).

Felix



More information about the wine-devel mailing list