[try2] kernel32: Add partial stub for
NeedCurrentDirectoryForExePath - what's wrong?
Kirill K. Smirnov
lich at math.spbu.ru
Wed Apr 25 00:13:10 CDT 2007
> On 4/24/07, Kirill K. Smirnov <lich at math.spbu.ru> wrote:
> > Hi,
> > I've sent the patch:
> > http://www.winehq.com/pipermail/wine-patches/2007-April/038205.html
> > about a week ago and it has not been applied. Please, explain, what is
> > wrong with it?
> You haven't sent in any tests for this function. Is there some app
> that needs it?
I've mentioned it in the first try: windows 2003 cmd.exe.
> + static const WCHAR env_name =
> That's a really long line. We try to keep the lines to a length of 80.
Hm... I've improve it.
> + /* Check name first */
> + if (strchrW(name, '\\'))
> + return TRUE;
> You should probably check name for NULL, but you'll have to test how
> Windows handles this. That's a pretty bad comment too; check name for
For containing backshash. Yes, comment must be better.
More information about the wine-devel