[ros-dev] [ros-diffs] [ekohl] 23521: - Implement ScmStartServiceA and revert the last modification of StartServiceA. - Fix EnumServicesStatusExW.
Hervé Poussineau
hpoussin at reactos.org
Tue Aug 8 12:16:18 CEST 2006
Hi,
> Subject: [ros-diffs] [ekohl] 23521:
> - Implement ScmStartServiceA and revert the last modification of
StartServiceA.
> - Fix EnumServicesStatusExW.
We now have an exact code duplication between StartServiceA/StartServiceW
(except strlen/wcslen and strcpy/wcscpy) and
ScmrStartServiceA/ScmrStartServiceW.
I don't think it's a good thing because we will have some problems to keep
the code in sync.
I did r23519 to prevent this code duplication and because
ScmrStartServiceA() was not called except in StartServiceA.
By the way, I also implemented argument conversion between ANSI and Unicode
in StartServiceA before calling StartServiceW, which meant that arguments
were ignored only in ScmrStartServiceW. Now, you have 2 places to take care
of them (ScmrStartServiceA/W)...
That was just to say my feeling...
Hervé
More information about the Ros-dev
mailing list