fix against loosing chat sessions during chat plugin's unload

git-svn-id: http://svn.miranda-ng.org/main/trunk@7746 1316c22d-e87f-b044-9b9b-93d7a3e3ba9c

6 files changed, 129 insertions, 117 deletions

diff --git a/plugins/Scriver/src/chat/main.cpp b/plugins/Scriver/src/chat/main.cpp
index b283bbec90..c5a3c2d8cb 100644
--- a/plugins/Scriver/src/chat/main.cpp
+++ b/plugins/Scriver/src/chat/main.cpp
@@ -201,7 +201,6 @@ int Chat_Unload(void)
 	db_set_w(NULL, "Chat", "SplitterX", (WORD)g_Settings.iSplitterX);

 

 	DestroyMenu(g_hMenu);

-	*pci = saveCI;

 	return 0;

 }

 

diff --git a/plugins/TabSRMM/src/chat/main.cpp b/plugins/TabSRMM/src/chat/main.cpp
index 7e3137bafc..57db8abc38 100644
--- a/plugins/TabSRMM/src/chat/main.cpp
+++ b/plugins/TabSRMM/src/chat/main.cpp
@@ -293,7 +293,6 @@ int Chat_Load()
 

 int Chat_Unload(void)

 {

-	*pci = saveCI;

 	DestroyMenu(g_hMenu);

 	return 0;

 }

diff --git a/src/core/stdchat/src/main.cpp b/src/core/stdchat/src/main.cpp
index b785eb0e46..81702484c2 100644
--- a/src/core/stdchat/src/main.cpp
+++ b/src/core/stdchat/src/main.cpp
@@ -364,7 +364,6 @@ extern "C" __declspec(dllexport) int Unload(void)
 	db_set_dw(NULL, "Chat", "roomheight", g_Settings.iHeight);

 

 	DestroyMenu(g_hMenu);

-	*pci = saveCI;

 	return 0;

 }

 

diff --git a/src/modules/chat/chat.h b/src/modules/chat/chat.h
index 6121d302f2..6fdfca7202 100644
--- a/src/modules/chat/chat.h
+++ b/src/modules/chat/chat.h
@@ -39,6 +39,7 @@ extern HGENMENU hJoinMenuItem, hLeaveMenuItem;
 extern GlobalLogSettingsBase *g_Settings;

 extern int g_cbSession, g_cbModuleInfo;

 extern TCHAR *g_szFontGroup;

+extern CRITICAL_SECTION cs;

 

 // log.c

 void   LoadMsgLogBitmaps(void);

diff --git a/src/modules/chat/chat_svc.cpp b/src/modules/chat/chat_svc.cpp
index c0ab9d8116..49fa9cebc8 100644
--- a/src/modules/chat/chat_svc.cpp
+++ b/src/modules/chat/chat_svc.cpp
@@ -27,7 +27,7 @@ INT_PTR SvcGetChatManager(WPARAM, LPARAM);
 #include "chat.h"

 

 HGENMENU hJoinMenuItem, hLeaveMenuItem;

-CRITICAL_SECTION	cs;

+CRITICAL_SECTION cs;

 

 static HANDLE

    hServiceRegister = NULL,

@@ -571,19 +571,8 @@ static int ModulesLoaded(WPARAM wParam, LPARAM lParam)
 /////////////////////////////////////////////////////////////////////////////////////////

 // Service creation

 

-static bool bInited = false;

-

 int LoadChatModule(void)

 {

-	CreateServiceFunction("GChat/GetInterface", SvcGetChatManager);

-	return 0;

-}

-

-void InitChatModule(void)

-{

-	if (bInited)

-		return;

-

 	InitializeCriticalSection(&cs);

 

 	HookEvent(ME_SYSTEM_MODULESLOADED, ModulesLoaded);

@@ -601,20 +590,18 @@ void InitChatModule(void)
 	CreateServiceFunction("GChat/PrebuildMenuEvent", PrebuildContactMenuSvc);

 	CreateServiceFunction("GChat/JoinChat", JoinChat);

 	CreateServiceFunction("GChat/LeaveChat", LeaveChat);

+	CreateServiceFunction("GChat/GetInterface", SvcGetChatManager);

 

 	ci.hSendEvent = CreateHookableEvent(ME_GC_EVENT);

 	ci.hBuildMenuEvent = CreateHookableEvent(ME_GC_BUILDMENU);

 

 	HookEvent(ME_FONT_RELOAD, FontsChanged);

 	HookEvent(ME_SKIN2_ICONSCHANGED, IconsChanged);

-	bInited = true;

+	return 0;

 }

 

 void UnloadChatModule(void)

 {

-	if (!bInited)

-		return;

-

 	OptionsUnInit();

 	DeleteCriticalSection(&cs);

 

diff --git a/src/modules/chat/manager.cpp b/src/modules/chat/manager.cpp
index 9ecd8c73c7..1888a097e7 100644
--- a/src/modules/chat/manager.cpp
+++ b/src/modules/chat/manager.cpp
@@ -24,10 +24,10 @@ Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA  02111-1307, USA.
 

 #include "chat.h"

 

-void InitChatModule(void);

-

 #define WINDOWS_COMMANDS_MAX 30

 

+CHAT_MANAGER ci;

+

 MODULEINFO *m_ModList = 0;

 

 static void SetActiveSessionEx(SESSION_INFO *si)

@@ -1337,113 +1337,140 @@ static BOOL LM_RemoveAll(LOGINFO** ppLogListStart, LOGINFO** ppLogListEnd)
 	return TRUE;

 }

 

-CHAT_MANAGER ci =

-{

-	SetActiveSession,

-	SetActiveSessionEx,

-	GetActiveSession,

-	SM_AddSession,

-	SM_RemoveSession,

-	SM_FindSession,

-	SM_AddUser,

-	SM_ChangeUID,

-	SM_ChangeNick,

-	SM_RemoveUser,

-	SM_SetOffline,

-	SM_SetTabbedWindowHwnd,

-	SM_GetStatusIcon,

-	SM_SetStatus,

-	SM_SetStatusEx,

-	SM_SendUserMessage,

-	SM_AddStatus,

-	SM_AddEventToAllMatchingUID,

-	SM_AddEvent,

-	SM_SendMessage,

-	SM_PostMessage,

-	SM_BroadcastMessage,

-	SM_RemoveAll,

-	SM_GiveStatus,

-	SM_SetContactStatus,

-	SM_TakeStatus,

-	SM_MoveUser,

-	SM_AddCommand,

-	SM_GetPrevCommand,

-	SM_GetNextCommand,

-	SM_GetCount,

-	SM_FindSessionByIndex,

-	SM_GetUsers,

-	SM_GetUserFromIndex,

-	SM_InvalidateLogDirectories,

-

-	MM_AddModule,

-	MM_FindModule,

-	MM_FixColors,

-	MM_FontsChanged,

-	MM_IconsChanged,

-	MM_RemoveAll,

-

-	TM_AddStatus,

-	TM_FindStatus,

-	TM_StringToWord,

-	TM_WordToString,

-	TM_RemoveAll,

-

-	UM_SetStatusEx,

-	UM_AddUser,

-	UM_SortUser,

-	UM_FindUser,

-	UM_FindUserFromIndex,

-	UM_GiveStatus,

-	UM_SetContactStatus,

-	UM_TakeStatus,

-	UM_FindUserAutoComplete,

-	UM_RemoveUser,

-	UM_RemoveAll,

-	UM_CompareItem,

-

-	LM_AddEvent,

-	LM_TrimLog,

-	LM_RemoveAll,

-

-	AddRoom,

-	SetOffline,

-	SetAllOffline,

-	AddEvent,

-	FindRoom,

-	

-	Log_CreateRTF,

-	Log_CreateRtfHeader,

-	LoadMsgDlgFont,

-	MakeTimeStamp,

-	

-	DoEventHook,

-	DoEventHookAsync,

-

-	DoSoundsFlashPopupTrayStuff,

-	DoTrayIcon,

-	DoPopup,

-	ShowPopup,

-	LogToFile,

-	GetChatLogsFilename,

-

-	IsHighlighted,

-	RemoveFormatting,

-	LoadGlobalSettings

-};

-

 INT_PTR SvcGetChatManager(WPARAM, LPARAM lParam)

 {

-	InitChatModule();

-

 	// wipe out old junk

 	memset(PBYTE(&ci) + offsetof(CHAT_MANAGER, OnCreateModule), 0, sizeof(CHAT_MANAGER)-offsetof(CHAT_MANAGER, OnCreateModule));

 

 	CHAT_MANAGER_INITDATA *pInit = (CHAT_MANAGER_INITDATA*)lParam;

+	if (g_cbSession) { // reallocate old sessions

+		mir_cslock lck(cs);

+		SESSION_INFO *pPrev = NULL;

+		for (SESSION_INFO *p = ci.wndList; p; p = p->next) {

+			SESSION_INFO *p1 = (SESSION_INFO*)mir_realloc(p, pInit->cbSession);

+			memset(PBYTE(p1) + sizeof(GCSessionInfoBase), 0, pInit->cbSession - sizeof(GCSessionInfoBase));

+			if (p1 != p) { // realloc could change a pointer, reinsert a structure

+				if (ci.wndList == p)

+					ci.wndList = p1;

+				if (pPrev != NULL)

+					pPrev->next = p1;

+				p = p1;

+			}

+			pPrev = p;

+		}

+	}

+	if (g_cbModuleInfo) { // reallocate old modules

+		mir_cslock lck(cs);

+		MODULEINFO *pPrev = NULL;

+		for (MODULEINFO *p = m_ModList; p; p = p->next) {

+			MODULEINFO *p1 = (MODULEINFO*)mir_realloc(p, pInit->cbModuleInfo);

+			memset(PBYTE(p1) + sizeof(GCModuleInfoBase), 0, pInit->cbModuleInfo - sizeof(GCModuleInfoBase));

+			if (p1 != p) { // realloc could change a pointer, reinsert a structure

+				if (m_ModList == p)

+					m_ModList = p1;

+				if (pPrev != NULL)

+					pPrev->next = p1;

+				p = p1;

+			}

+			pPrev = p;

+		}

+	}

 	g_Settings = pInit->pSettings;

 	g_szFontGroup = pInit->szFontGroup;

 	g_cbSession = pInit->cbSession;

 	g_cbModuleInfo = pInit->cbModuleInfo;

 

+	ci.SetActiveSession = SetActiveSession;

+	ci.SetActiveSessionEx = SetActiveSessionEx;

+	ci.GetActiveSession = GetActiveSession;

+	ci.SM_AddSession = SM_AddSession;

+	ci.SM_RemoveSession = SM_RemoveSession;

+	ci.SM_FindSession = SM_FindSession;

+	ci.SM_AddUser = SM_AddUser;

+	ci.SM_ChangeUID = SM_ChangeUID;

+	ci.SM_ChangeNick = SM_ChangeNick;

+	ci.SM_RemoveUser = SM_RemoveUser;

+	ci.SM_SetOffline = SM_SetOffline;

+	ci.SM_SetTabbedWindowHwnd = SM_SetTabbedWindowHwnd;

+	ci.SM_GetStatusIcon = SM_GetStatusIcon;

+	ci.SM_SetStatus = SM_SetStatus;

+	ci.SM_SetStatusEx = SM_SetStatusEx;

+	ci.SM_SendUserMessage = SM_SendUserMessage;

+	ci.SM_AddStatus = SM_AddStatus;

+	ci.SM_AddEventToAllMatchingUID = SM_AddEventToAllMatchingUID;

+	ci.SM_AddEvent = SM_AddEvent;

+	ci.SM_SendMessage = SM_SendMessage;

+	ci.SM_PostMessage = SM_PostMessage;

+	ci.SM_BroadcastMessage = SM_BroadcastMessage;

+	ci.SM_RemoveAll = SM_RemoveAll;

+	ci.SM_GiveStatus = SM_GiveStatus;

+	ci.SM_SetContactStatus = SM_SetContactStatus;

+	ci.SM_TakeStatus = SM_TakeStatus;

+	ci.SM_MoveUser = SM_MoveUser;

+	ci.SM_AddCommand = SM_AddCommand;

+	ci.SM_GetPrevCommand = SM_GetPrevCommand;

+	ci.SM_GetNextCommand = SM_GetNextCommand;

+	ci.SM_GetCount = SM_GetCount;

+	ci.SM_FindSessionByIndex = SM_FindSessionByIndex;

+	ci.SM_GetUsers = SM_GetUsers;

+	ci.SM_GetUserFromIndex = SM_GetUserFromIndex;

+	ci.SM_InvalidateLogDirectories = SM_InvalidateLogDirectories;

+	

+	ci.MM_AddModule = MM_AddModule;

+	ci.MM_FindModule = MM_FindModule;

+	ci.MM_FixColors = MM_FixColors;

+	ci.MM_FontsChanged = MM_FontsChanged;

+	ci.MM_IconsChanged = MM_IconsChanged;

+	ci.MM_RemoveAll = MM_RemoveAll;

+	

+	ci.TM_AddStatus = TM_AddStatus;

+	ci.TM_FindStatus = TM_FindStatus;

+	ci.TM_StringToWord = TM_StringToWord;

+	ci.TM_WordToString = TM_WordToString;

+	ci.TM_RemoveAll = TM_RemoveAll;

+

+	ci.UM_SetStatusEx = UM_SetStatusEx;

+	ci.UM_AddUser = UM_AddUser;

+	ci.UM_SortUser = UM_SortUser;

+	ci.UM_FindUser = UM_FindUser;

+	ci.UM_FindUserFromIndex = UM_FindUserFromIndex;

+	ci.UM_GiveStatus = UM_GiveStatus;

+	ci.UM_SetContactStatus = UM_SetContactStatus;

+	ci.UM_TakeStatus = UM_TakeStatus;

+	ci.UM_FindUserAutoComplete = UM_FindUserAutoComplete;

+	ci.UM_RemoveUser = UM_RemoveUser;

+	ci.UM_RemoveAll = UM_RemoveAll;

+	ci.UM_CompareItem = UM_CompareItem;

+	

+	ci.LM_AddEvent = LM_AddEvent;

+	ci.LM_TrimLog = LM_TrimLog;

+	ci.LM_RemoveAll = LM_RemoveAll;

+

+	ci.AddRoom = AddRoom;

+	ci.SetOffline = SetOffline;

+	ci.SetAllOffline = SetAllOffline;

+	ci.AddEvent = AddEvent;

+	ci.FindRoom = FindRoom;

+	

+	ci.Log_CreateRTF = Log_CreateRTF;

+	ci.Log_CreateRtfHeader = Log_CreateRtfHeader;

+	ci.LoadMsgDlgFont = LoadMsgDlgFont;

+	ci.MakeTimeStamp = MakeTimeStamp;

+	

+	ci.DoEventHook = DoEventHook;

+	ci.DoEventHookAsync = DoEventHookAsync;

+	

+	ci.DoSoundsFlashPopupTrayStuff = DoSoundsFlashPopupTrayStuff;

+	ci.DoTrayIcon = DoTrayIcon;

+	ci.DoPopup = DoPopup;

+	ci.ShowPopup = ShowPopup;

+	ci.LogToFile = LogToFile;

+	ci.GetChatLogsFilename = GetChatLogsFilename;

+

+	ci.IsHighlighted = IsHighlighted;

+	ci.RemoveFormatting = RemoveFormatting;

+	ci.ReloadSettings = LoadGlobalSettings;

+

 	LoadChatIcons();

 	RegisterFonts();

 	OptionsInit();