shell shorcuts

[Martin Fuchs]

Hello,

On 06.01.2004 18:51:16 Subhobroto  Sinha wrote:
>  I had completed the patch for IPersistFile::Load() and associated functions.
>  In the last patch which I had submitted for review here, Mike McCormack had
>  pointed out an incorrect assumption that I had made about all locales being
>  DBCS. Thanks to him, many of you will not suffer ;)

I am working on the ReactOS Explorer, which uses Wine's shell32 implementation.
We already have made many bugfixes and estensions to the shell link and other
shell32 code. One of those extensions is the implementation of control panel in
shell namespace. We will post this big patch for shell32 next time.
(when some remaining problems will be solved)

As I saw in from the comment in your Stream_LoadString() patch, you are not
sure about the meaning of the SCF_UNICODE flag in *.lnk files.
Well, I think it's because it has been defined wrong. I am currently using this
definition:

#define SCF_UNICODE 0x80        (instead of 0x1000)

This is at least compatible to links of WIN95 (ANSI strings) and WIN XP (UTF16 strings).
I did not yet test any other operating system versions. With this corrected
defintion and a few more extensions Stream_LoadString() works pretty well
on ReactOS.

Regards,

   Martin