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
ddiego
post Feb 18 2008, 04:18 PM
Post #2


VCF Lead Developer
***

Group: Lead Developer
Posts: 2834
Joined: 2-August 03
Member No.: 2



I would go for your first choice, just cause it "looks" better, but that's just me.
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 : 18th May 2013 - 06:32 PM