IPB SourceForge.net Logo

Welcome Guest ( Log In | Register )

[ Outline ] · Standard · Linear+

> Recent Win32UIShell.cpp changes

Fraggle
post Feb 18 2008, 01:32 AM
Post #1


Advanced Member
***

Group: Members
Posts: 518
Joined: 6-December 04
Member No.: 112



This file's been throwing me since it was altered last week. Couple of problems I think I've found:

1) The calls to SHEmptyRecycleBin are leaving me a bit confused. MSDN claims that the function is valid as far back as Win95 with IE4. However, my version of shellapi.h has it ifdeffed for NTDDI_VERSION >= NTDDI_WIN2K. I'm not sure what MS is playing at here, or if you know something I don't.

2) VC2005 is complaining about line 454

ext.insert( 0, '.' );

saying it can't convert from char to String. The relevant String declarations for insert are

UnicodeString& insert(size_type p0, const AnsiChar *s);

UnicodeString& insert(size_type p0, size_type n, AnsiChar c);

so I guess the choice is

ext.insert( 0, ".");
or
ext.insert( 0, 1, '.'); // at least I think it would be 1

Is the second form quicker as it doesn't need to worry about null-termination?


--------------------
FLAGRANT SYSTEM ERROR
Computer Over.
Virus = Very Yes.
User is offlineProfile CardPM
Go to the top of the page
+Quote Post
 
Reply to this topicStart new topicStart Poll
Replies
Fraggle
post Feb 18 2008, 05:37 PM
Post #2


Advanced Member
***

Group: Members
Posts: 518
Joined: 6-December 04
Member No.: 112



Yeah - using VS2005, and it does throw an error. The temp fix is to set WINVER and _WIN32_WINNT to 0x0500 in FrameworkConfig.h

My SDK version is from VS2008. I haven't tried it yet, but I can't see why it won't fail there either.

I think the first one 'looks' better too. :-) Might as well go with it.


--------------------
FLAGRANT SYSTEM ERROR
Computer Over.
Virus = Very Yes.
User is offlineProfile CardPM
Go to the top of the page
+Quote Post

Posts in this topic


Reply to this topicTopic OptionsStart new topic
1 User(s) are reading this topic (1 Guests and 0 Anonymous Users)
0 Members:
 

Lo-fi Version : 19th May 2013 - 02:00 AM