Some code clean-up.

This commit is contained in:
LoRd_MuldeR 2013-11-29 03:48:11 +01:00
parent 1d8709689b
commit 5e9d8bf850
3 changed files with 61 additions and 85 deletions

View File

@ -35,7 +35,7 @@
#define VER_LAMEXP_MINOR_LO 9 #define VER_LAMEXP_MINOR_LO 9
#define VER_LAMEXP_TYPE Alpha #define VER_LAMEXP_TYPE Alpha
#define VER_LAMEXP_PATCH 8 #define VER_LAMEXP_PATCH 8
#define VER_LAMEXP_BUILD 1489 #define VER_LAMEXP_BUILD 1490
#define VER_LAMEXP_CONFG 1348 #define VER_LAMEXP_CONFG 1348
/////////////////////////////////////////////////////////////////////////////// ///////////////////////////////////////////////////////////////////////////////

View File

@ -85,7 +85,7 @@ QString lamexp_rand_str(const bool bLong)
*/ */
static QString lamexp_try_init_folder(const QString &folderPath) static QString lamexp_try_init_folder(const QString &folderPath)
{ {
static const char *DATA = "Lorem ipsum dolor sit amet, consectetur, adipisci velit!"; static const char *TEST_DATA = "Lorem ipsum dolor sit amet, consectetur, adipisci velit!";
bool success = false; bool success = false;
@ -93,17 +93,16 @@ static QString lamexp_try_init_folder(const QString &folderPath)
const QDir folderDir(folderInfo.absoluteFilePath()); const QDir folderDir(folderInfo.absoluteFilePath());
//Create folder, if it does *not* exist yet //Create folder, if it does *not* exist yet
for(int i = 0; i < 16; i++) for(int i = 0; (i < 16) && (!folderDir.exists()); i++)
{ {
if(folderDir.exists()) break;
folderDir.mkpath("."); folderDir.mkpath(".");
} }
//Make sure folder exists now *and* is writable //Make sure folder exists now *and* is writable
if(folderDir.exists()) if(folderDir.exists())
{ {
const QByteArray testData = QByteArray(DATA); const QByteArray testData = QByteArray(TEST_DATA);
for(int i = 0; i < 32; i++) for(int i = 0; (i < 32) && (!success); i++)
{ {
QFile testFile(folderDir.absoluteFilePath(QString("~%1.tmp").arg(lamexp_rand_str()))); QFile testFile(folderDir.absoluteFilePath(QString("~%1.tmp").arg(lamexp_rand_str())));
if(testFile.open(QIODevice::ReadWrite | QIODevice::Truncate)) if(testFile.open(QIODevice::ReadWrite | QIODevice::Truncate))
@ -113,9 +112,7 @@ static QString lamexp_try_init_folder(const QString &folderPath)
success = true; success = true;
} }
testFile.remove(); testFile.remove();
testFile.close();
} }
if(success) break;
} }
} }

View File

@ -119,6 +119,13 @@ while(0)
} \ } \
while(0) while(0)
///////////////////////////////////////////////////////////////////////////////
// GLOBAL TYPES
///////////////////////////////////////////////////////////////////////////////
typedef HRESULT (WINAPI *SHGetKnownFolderPath_t)(const GUID &rfid, DWORD dwFlags, HANDLE hToken, PWSTR *ppszPath);
typedef HRESULT (WINAPI *SHGetFolderPath_t)(HWND hwndOwner, int nFolder, HANDLE hToken, DWORD dwFlags, LPWSTR pszPath);
/////////////////////////////////////////////////////////////////////////////// ///////////////////////////////////////////////////////////////////////////////
// GLOBAL VARS // GLOBAL VARS
/////////////////////////////////////////////////////////////////////////////// ///////////////////////////////////////////////////////////////////////////////
@ -130,6 +137,8 @@ static bool g_lamexp_console_attached = false;
static struct static struct
{ {
QMap<size_t, QString> *knownFolders; QMap<size_t, QString> *knownFolders;
SHGetKnownFolderPath_t getKnownFolderPath;
SHGetFolderPath_t getFolderPath;
QReadWriteLock lock; QReadWriteLock lock;
} }
g_lamexp_known_folder; g_lamexp_known_folder;
@ -1152,60 +1161,46 @@ const QStringList &lamexp_arguments(void)
*/ */
const QString &lamexp_known_folder(lamexp_known_folder_t folder_id) const QString &lamexp_known_folder(lamexp_known_folder_t folder_id)
{ {
typedef HRESULT (WINAPI *SHGetKnownFolderPathFun)(__in const GUID &rfid, __in DWORD dwFlags, __in HANDLE hToken, __out PWSTR *ppszPath); static const int CSIDL_FLAG_CREATE = 0x8000;
typedef HRESULT (WINAPI *SHGetFolderPathFun)(__in HWND hwndOwner, __in int nFolder, __in HANDLE hToken, __in DWORD dwFlags, __out LPWSTR pszPath); typedef enum { KF_FLAG_CREATE = 0x00008000 } kf_flags_t;
struct
{
const int csidl;
const GUID guid;
}
static s_folders[] =
{
{ 0x001c, {0xF1B32785,0x6FBA,0x4FCF,{0x9D,0x55,0x7B,0x8E,0x7F,0x15,0x70,0x91}} }, //CSIDL_LOCAL_APPDATA
{ 0x0026, {0x905e63b6,0xc1bf,0x494e,{0xb2,0x9c,0x65,0xb7,0x32,0xd3,0xd2,0x1a}} }, //CSIDL_PROGRAM_FILES
{ 0x0024, {0xF38BF404,0x1D43,0x42F2,{0x93,0x05,0x67,0xDE,0x0B,0x28,0xFC,0x23}} }, //CSIDL_WINDOWS_FOLDER
{ 0x0025, {0x1AC14E77,0x02E7,0x4E5D,{0xB7,0x44,0x2E,0xB1,0xAE,0x51,0x98,0xB7}} }, //CSIDL_SYSTEM_FOLDER
};
static const int CSIDL_LOCAL_APPDATA = 0x001c; size_t folderId = size_t(-1);
static const int CSIDL_PROGRAM_FILES = 0x0026;
static const int CSIDL_WINDOWS_FOLDER = 0x0024;
static const int CSIDL_SYSTEM_FOLDER = 0x0025;
static const GUID GUID_LOCAL_APPDATA = {0xF1B32785,0x6FBA,0x4FCF,{0x9D,0x55,0x7B,0x8E,0x7F,0x15,0x70,0x91}};
static const GUID GUID_LOCAL_APPDATA_LOW = {0xA520A1A4,0x1780,0x4FF6,{0xBD,0x18,0x16,0x73,0x43,0xC5,0xAF,0x16}};
static const GUID GUID_PROGRAM_FILES = {0x905e63b6,0xc1bf,0x494e,{0xb2,0x9c,0x65,0xb7,0x32,0xd3,0xd2,0x1a}};
static const GUID GUID_WINDOWS_FOLDER = {0xF38BF404,0x1D43,0x42F2,{0x93,0x05,0x67,0xDE,0x0B,0x28,0xFC,0x23}};
static const GUID GUID_SYSTEM_FOLDER = {0x1AC14E77,0x02E7,0x4E5D,{0xB7,0x44,0x2E,0xB1,0xAE,0x51,0x98,0xB7}};
QReadLocker readLock(&g_lamexp_known_folder.lock);
int folderCSIDL = -1;
GUID folderGUID = {0x0000,0x0000,0x0000,{0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00}};
size_t folderCacheId = size_t(-1);
switch(folder_id) switch(folder_id)
{ {
case lamexp_folder_localappdata: case lamexp_folder_localappdata: folderId = 0; break;
folderCacheId = 0; case lamexp_folder_programfiles: folderId = 1; break;
folderCSIDL = CSIDL_LOCAL_APPDATA; case lamexp_folder_systroot_dir: folderId = 2; break;
folderGUID = GUID_LOCAL_APPDATA; case lamexp_folder_systemfolder: folderId = 3; break;
break; }
case lamexp_folder_programfiles:
folderCacheId = 1; if(folderId == size_t(-1))
folderCSIDL = CSIDL_PROGRAM_FILES; {
folderGUID = GUID_PROGRAM_FILES;
break;
case lamexp_folder_systemfolder:
folderCacheId = 2;
folderCSIDL = CSIDL_SYSTEM_FOLDER;
folderGUID = GUID_SYSTEM_FOLDER;
break;
case lamexp_folder_systroot_dir:
folderCacheId = 3;
folderCSIDL = CSIDL_WINDOWS_FOLDER;
folderGUID = GUID_WINDOWS_FOLDER;
break;
default:
qWarning("Invalid 'known' folder was requested!"); qWarning("Invalid 'known' folder was requested!");
return *reinterpret_cast<QString*>(NULL); return *reinterpret_cast<QString*>(NULL);
break;
} }
QReadLocker readLock(&g_lamexp_known_folder.lock);
//Already in cache? //Already in cache?
if(g_lamexp_known_folder.knownFolders) if(g_lamexp_known_folder.knownFolders)
{ {
if(g_lamexp_known_folder.knownFolders->contains(folderCacheId)) if(g_lamexp_known_folder.knownFolders->contains(folderId))
{ {
return (*g_lamexp_known_folder.knownFolders)[folderCacheId]; return (*g_lamexp_known_folder.knownFolders)[folderId];
} }
} }
@ -1216,75 +1211,59 @@ const QString &lamexp_known_folder(lamexp_known_folder_t folder_id)
//Still not in cache? //Still not in cache?
if(g_lamexp_known_folder.knownFolders) if(g_lamexp_known_folder.knownFolders)
{ {
if(g_lamexp_known_folder.knownFolders->contains(folderCacheId)) if(g_lamexp_known_folder.knownFolders->contains(folderId))
{ {
return (*g_lamexp_known_folder.knownFolders)[folderCacheId]; return (*g_lamexp_known_folder.knownFolders)[folderId];
} }
} }
static SHGetKnownFolderPathFun SHGetKnownFolderPathPtr = NULL; //Initialize on first call
static SHGetFolderPathFun SHGetFolderPathPtr = NULL; if(!g_lamexp_known_folder.knownFolders)
//Lookup functions
if((!SHGetKnownFolderPathPtr) && (!SHGetFolderPathPtr))
{ {
QLibrary kernel32Lib("shell32.dll"); QLibrary shell32("shell32.dll");
if(kernel32Lib.load()) if(shell32.load())
{ {
SHGetKnownFolderPathPtr = (SHGetKnownFolderPathFun) kernel32Lib.resolve("SHGetKnownFolderPath"); g_lamexp_known_folder.getFolderPath = (SHGetFolderPath_t) shell32.resolve("SHGetFolderPathW");
SHGetFolderPathPtr = (SHGetFolderPathFun) kernel32Lib.resolve("SHGetFolderPathW"); g_lamexp_known_folder.getKnownFolderPath = (SHGetKnownFolderPath_t) shell32.resolve("SHGetKnownFolderPath");
} }
g_lamexp_known_folder.knownFolders = new QMap<size_t, QString>();
} }
QString folder; QString folderPath;
//Now try to get the folder path! //Now try to get the folder path!
if(SHGetKnownFolderPathPtr) if(g_lamexp_known_folder.getKnownFolderPath)
{ {
WCHAR *path = NULL; WCHAR *path = NULL;
if(SHGetKnownFolderPathPtr(folderGUID, 0x00008000, NULL, &path) == S_OK) if(g_lamexp_known_folder.getKnownFolderPath(s_folders[folderId].guid, KF_FLAG_CREATE, NULL, &path) == S_OK)
{ {
//MessageBoxW(0, path, L"SHGetKnownFolderPath", MB_TOPMOST); //MessageBoxW(0, path, L"SHGetKnownFolderPath", MB_TOPMOST);
QDir folderTemp = QDir(QDir::fromNativeSeparators(QString::fromUtf16(reinterpret_cast<const unsigned short*>(path)))); QDir folderTemp = QDir(QDir::fromNativeSeparators(QString::fromUtf16(reinterpret_cast<const unsigned short*>(path))));
if(!folderTemp.exists())
{
folderTemp.mkpath(".");
}
if(folderTemp.exists()) if(folderTemp.exists())
{ {
folder = folderTemp.canonicalPath(); folderPath = folderTemp.canonicalPath();
} }
CoTaskMemFree(path); CoTaskMemFree(path);
} }
} }
else if(SHGetFolderPathPtr) else if(g_lamexp_known_folder.getFolderPath)
{ {
WCHAR *path = new WCHAR[4096]; WCHAR *path = new WCHAR[4096];
if(SHGetFolderPathPtr(NULL, folderCSIDL, NULL, NULL, path) == S_OK) if(g_lamexp_known_folder.getFolderPath(NULL, s_folders[folderId].csidl | CSIDL_FLAG_CREATE, NULL, NULL, path) == S_OK)
{ {
//MessageBoxW(0, path, L"SHGetFolderPathW", MB_TOPMOST); //MessageBoxW(0, path, L"SHGetFolderPathW", MB_TOPMOST);
QDir folderTemp = QDir(QDir::fromNativeSeparators(QString::fromUtf16(reinterpret_cast<const unsigned short*>(path)))); QDir folderTemp = QDir(QDir::fromNativeSeparators(QString::fromUtf16(reinterpret_cast<const unsigned short*>(path))));
if(!folderTemp.exists())
{
folderTemp.mkpath(".");
}
if(folderTemp.exists()) if(folderTemp.exists())
{ {
folder = folderTemp.canonicalPath(); folderPath = folderTemp.canonicalPath();
} }
} }
delete [] path; LAMEXP_DELETE_ARRAY(path);
}
//Create cache
if(!g_lamexp_known_folder.knownFolders)
{
g_lamexp_known_folder.knownFolders = new QMap<size_t, QString>();
} }
//Update cache //Update cache
g_lamexp_known_folder.knownFolders->insert(folderCacheId, folder); g_lamexp_known_folder.knownFolders->insert(folderId, folderPath);
return (*g_lamexp_known_folder.knownFolders)[folderCacheId]; return (*g_lamexp_known_folder.knownFolders)[folderId];
} }
/* /*