wine/dlls/shell32 shell32_main.c
Rein Klazes
rklazes at xs4all.nl
Fri Jul 4 12:46:52 CDT 2003
On Fri, 4 Jul 2003 18:31:16 +0100, you wrote:
> >| + /* psfi is NULL normally to query EXE type, if not none of the
> below makes
> >| + sense anyway. Windows allows this and just returns FALSE
> */
> >| + if (psfi != NULL) return FALSE;
>
> >this is obviously wrong (psfi is dereferenced in the remainder of the
> >function). I just wonder if it should be removed, or is here a typo and
> >should != be == ?
>
> Its a typo - I suspect I coded and tested it as if (!psfi) return false and
> 'tidied' it up incorrectly at the last minute. If should (obviously!) say
> "if (psfi == null) return FALSE;"
>
> Sincere apologies for this - repeat after me... I must re-test on the very
> final version of the patch....:-)
>
> Could I ask you to patch it though, as I will not get to do it until Tues
> otherwise (I'll do it then if you havent before).
Sure. I will fix the comment as well, I think it did describe the wrong
code accurately.
Rein.
--
Rein Klazes
rklazes at xs4all.nl
More information about the wine-devel
mailing list