Flummi/amxmodx
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Replace CLang file hashing with .st_mtime

Browse Source 
Replace CLang file hashing with .st_mtime for performance.
Also, fix a problem in CLangMngr::MergeDefinitionFile.
This commit is contained in:
HttrckCldHKS 2015-02-19 21:28:45 +02:00
parent 2b9e9c79b9
commit 530d0bf570
2 changed files with 13 additions and 56 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

60
amxmodx/CLang.cpp
View File

@ -318,48 +318,22 @@ void reparse_color(String* def)
// -- BAILOPAN // -- BAILOPAN
int CLangMngr::MergeDefinitionFile(const char *file) int CLangMngr::MergeDefinitionFile(const char *file)
{ {
const char* md5buffer = hashFile(file, Hash_Md5); /** Tries to open the file. */
if (!md5buffer) struct stat fileStat;
if (stat(file, &fileStat))
{ {
CVector<md5Pair *>::iterator iter; FileList.remove(file);
for (iter = FileList.begin(); iter != FileList.end(); ++iter)
{
if ((*iter)->file.compare(file) == 0)
{
char buf[33] = {0};
(*iter)->val.assign(buf);
break;
}
}
AMXXLOG_Log("[AMXX] Failed to open dictionary file: %s", file); AMXXLOG_Log("[AMXX] Failed to open dictionary file: %s", file);
return 0; return 0;
} }
bool foundFlag = false;
CVector<md5Pair *>::iterator iter;
for (iter = FileList.begin(); iter != FileList.end(); ++iter)
{
if ((*iter)->file.compare(file) == 0)
{
if ((*iter)->val.compare(md5buffer) == 0)
{
return -1;
} else {
(*iter)->val.assign(md5buffer);
break;
}
foundFlag = true;
}
}
if (!foundFlag) /** Checks if there is an existing entry with same time stamp. */
{ time_t timeStamp;
md5Pair *p = new md5Pair; if (FileList.retrieve(file, &timeStamp) && fileStat.st_mtime == timeStamp)
p->file.assign(file); return -1;
p->val.assign(md5buffer);
FileList.push_back(p); /** If yes, it either means that the entry doesn't exist or the existing entry needs to be updated. */
} FileList.replace(file, fileStat.st_mtime);
FILE* fp = fopen(file, "rt"); FILE* fp = fopen(file, "rt");
if (!fp) if (!fp)
@ -532,12 +506,6 @@ const char *CLangMngr::GetDef(const char *langName, const char *key, int &status
void CLangMngr::InvalidateCache() void CLangMngr::InvalidateCache()
{ {
for (size_t i = 0; i < FileList.size(); i++)
{
if (FileList[i])
delete FileList[i];
}
FileList.clear(); FileList.clear();
} }
@ -558,12 +526,6 @@ void CLangMngr::Clear()
delete m_Languages[i]; delete m_Languages[i];
} }
for (i = 0; i < FileList.size(); i++)
{
if (FileList[i])
delete FileList[i];
}
for (i = 0; i < KeyList.size(); i++) for (i = 0; i < KeyList.size(); i++)
{ {
if (KeyList[i]) if (KeyList[i])

9
amxmodx/CLang.h
View File

@ -11,6 +11,7 @@
#define _INCLUDE_CLANG_H #define _INCLUDE_CLANG_H
#include "sh_tinyhash.h" #include "sh_tinyhash.h"
#include "sm_stringhashmap.h"
#define LANG_SERVER 0 #define LANG_SERVER 0
#define LANG_PLAYER -1 #define LANG_PLAYER -1
@ -18,12 +19,6 @@
#define ERR_BADKEY 1 // Lang key not found #define ERR_BADKEY 1 // Lang key not found
#define ERR_BADLANG 2 // Invalid lang #define ERR_BADLANG 2 // Invalid lang
struct md5Pair
{
String file;
String val;
};
struct sKeyDef struct sKeyDef
{ {
String *definition; String *definition;
@ -118,7 +113,7 @@ private:
LangVec m_Languages; LangVec m_Languages;
CVector<md5Pair *> FileList; StringHashMap<time_t> FileList;
CVector<String *> KeyList; CVector<String *> KeyList;
THash<String, keytbl_val> KeyTable; THash<String, keytbl_val> KeyTable;