--remove
@@ -20,6 +20,6 @@ | ||
20 | 20 | #define NMS_ADDIN_STRRSC_SHOWOPTIONDIALOG NMD_ADDIN_STRRSC TEXT("/ShowOptionDialog") |
21 | 21 | |
22 | 22 | // 言語変更通知用イベント |
23 | -#define STRRSC_EVENT_CHANGELANGUAGE NMM_ADDIN TEXT("/") NMP_ADDIN_NAME TEXT("/OnChangeLanguage") | |
23 | +#define NME_ADDIN_STRRSC_CHANGELANGUAGE NMD_ADDIN_STRRSC TEXT("/OnChangeLanguage") | |
24 | 24 | |
25 | 25 | #endif |
@@ -43,11 +43,11 @@ | ||
43 | 43 | CreateNsmService(NMS_ADDIN_STRRSC_GETLOCALSTR, StrRscService::DoGetLocalStr); |
44 | 44 | CreateNsmService(NMS_ADDIN_STRRSC_SHOWOPTIONDIALOG, StrRscService::DoShowOptionDialog); |
45 | 45 | |
46 | - ChangeLangEvent = CreateNsmEvent(STRRSC_EVENT_CHANGELANGUAGE); | |
46 | + ChangeLangEvent = CreateNsmEvent(NME_ADDIN_STRRSC_CHANGELANGUAGE); | |
47 | 47 | } |
48 | 48 | |
49 | 49 | // オプションダイアログ表示 |
50 | - INT_PTR StrRscMain::DoShowOptionDialog(const HWND parent) | |
50 | + INT_PTR StrRscMain::DoShowOptionDialog(HWND parent) | |
51 | 51 | { |
52 | 52 | OptionDlg dlg; |
53 | 53 | dlg.Locale = Config.Locale; |
@@ -76,14 +76,20 @@ | ||
76 | 76 | + pluginName + TEXT(".txt"); |
77 | 77 | |
78 | 78 | if(GetFileAttributes(filePath.c_str()) != INVALID_FILE_ATTRIBUTES) |
79 | - IniFiles.insert(std::make_pair(pluginName,filePath)); | |
79 | + IniFiles.insert(std::make_pair(pluginName, filePath)); | |
80 | 80 | } |
81 | 81 | |
82 | 82 | // 言語ファイルから文字列取得 |
83 | - int StrRscMain::DoGetLocalStr(const PNsmStrRsc strRsc) | |
83 | + int StrRscMain::DoGetLocalStr(const NsmStrRsc &strRsc) | |
84 | 84 | { |
85 | - const std::tstring pluginName = c2t(strRsc->lpPluginName); | |
85 | + if(!strRsc.lpDefaultStr || !strRsc.lpMessageTag || !strRsc.lpPluginName || !strRsc.lpSectionName) | |
86 | + return 0; | |
86 | 87 | |
88 | + if(strRsc.nBufferSize < 1) | |
89 | + return 0; | |
90 | + | |
91 | + const std::tstring pluginName = c2t(strRsc.lpPluginName); | |
92 | + | |
87 | 93 | IniFilesTable::const_iterator it = IniFiles.find(pluginName); |
88 | 94 | |
89 | 95 | if(it == IniFiles.end()) |
@@ -93,36 +99,41 @@ | ||
93 | 99 | |
94 | 100 | if(it == IniFiles.end()) |
95 | 101 | { |
96 | - strRsc->nBufferSize = StrToStrRsc(strRsc,c2t(strRsc->lpDefaultStr)); | |
102 | + StrToStrRsc(strRsc, c2t(strRsc.lpDefaultStr)); | |
103 | + | |
97 | 104 | return 0; |
98 | 105 | } |
99 | 106 | } |
100 | 107 | |
101 | - std::vector<TCHAR> buff(strRsc->nBufferSize); | |
108 | + std::vector<TCHAR> buff(strRsc.nBufferSize); | |
102 | 109 | GetPrivateProfileString( |
103 | - c2t(strRsc->lpSectionName).c_str(), | |
104 | - c2t(strRsc->lpMessageTag).c_str(), | |
105 | - c2t(strRsc->lpDefaultStr).c_str(), | |
106 | - &buff[0],static_cast<DWORD>(buff.size()),(*it).second.c_str()); | |
110 | + c2t(strRsc.lpSectionName).c_str(), | |
111 | + c2t(strRsc.lpMessageTag).c_str(), | |
112 | + c2t(strRsc.lpDefaultStr).c_str(), | |
113 | + &buff[0], static_cast<DWORD>(buff.size()), (*it).second.c_str()); | |
107 | 114 | |
108 | - StrToStrRsc(strRsc, &buff[0]); | |
115 | + if(!StrToStrRsc(strRsc, &buff[0])) | |
116 | + return 0; | |
109 | 117 | |
110 | 118 | return 1; |
111 | 119 | } |
112 | 120 | |
113 | 121 | // String->StrRsc 変換関数 |
114 | - int StrRscMain::StrToStrRsc(const PNsmStrRsc strRsc,const std::tstring &source) | |
122 | + int StrRscMain::StrToStrRsc(const NsmStrRsc &strRsc, const std::tstring &source) | |
115 | 123 | { |
116 | - if(strRsc->nBufferSize < 1) | |
124 | + if(!strRsc.lpBuffer) | |
117 | 125 | return 0; |
118 | 126 | |
127 | + if(strRsc.nBufferSize < 1) | |
128 | + return 0; | |
129 | + | |
119 | 130 | std::string tmp = t2c(source); |
120 | 131 | |
121 | - std::string::size_type len = std::min<std::string::size_type>(strRsc->nBufferSize-1,tmp.size()); | |
132 | + std::string::size_type len = std::min<std::string::size_type>(strRsc.nBufferSize - 1, tmp.size()); | |
122 | 133 | |
123 | 134 | tmp.resize(len); |
124 | 135 | |
125 | - strcpy_s(strRsc->lpBuffer,strRsc->nBufferSize,tmp.c_str()); | |
136 | + strcpy_s(strRsc.lpBuffer, strRsc.nBufferSize, tmp.c_str()); | |
126 | 137 | |
127 | 138 | return static_cast<int>(len); |
128 | 139 | } |
@@ -138,13 +149,16 @@ | ||
138 | 149 | |
139 | 150 | LONG_PTR __stdcall StrRscService::DoGetLocalStr(const WPARAM wParam, const LPARAM /* lParam */) |
140 | 151 | { |
141 | - return g_strRscMain.DoGetLocalStr(reinterpret_cast<PNsmStrRsc>(wParam)); | |
152 | + if(!wParam) | |
153 | + return 0; | |
154 | + | |
155 | + return g_strRscMain.DoGetLocalStr(*reinterpret_cast<NsmStrRsc*>(wParam)); | |
142 | 156 | } |
143 | 157 | } |
144 | 158 | } |
145 | 159 | } |
146 | 160 | |
147 | -BOOL APIENTRY DllMain(HINSTANCE hinstDLL,DWORD fdwReason,LPVOID /*lpvReserved*/) | |
161 | +BOOL APIENTRY DllMain(HINSTANCE hinstDLL, DWORD fdwReason, LPVOID /*lpvReserved*/) | |
148 | 162 | { |
149 | 163 | g_hInstance = hinstDLL; |
150 | 164 |
@@ -173,7 +187,7 @@ | ||
173 | 187 | // |
174 | 188 | // Return : 実際に書き込んだ情報のサイズ |
175 | 189 | // nInfoNo が無効なら 0 |
176 | - int WINAPI GetPluginInfo(const int nInfNo, LPSTR lpBuffer,const int nSize) | |
190 | + int WINAPI GetPluginInfo(const int nInfNo, LPSTR lpBuffer, const int nSize) | |
177 | 191 | { |
178 | 192 | if(!lpBuffer) |
179 | 193 | return 0; |
@@ -186,10 +200,10 @@ | ||
186 | 200 | if(result.empty()) |
187 | 201 | return 0; |
188 | 202 | |
189 | - std::string::size_type len = std::min<std::string::size_type>(nSize-1,result.size()); | |
203 | + std::string::size_type len = std::min<std::string::size_type>(nSize-1, result.size()); | |
190 | 204 | |
191 | 205 | result.resize(len); |
192 | - strcpy_s(lpBuffer,nSize,result.c_str()); | |
206 | + strcpy_s(lpBuffer, nSize, result.c_str()); | |
193 | 207 | |
194 | 208 | return static_cast<int>(len); |
195 | 209 | } |
@@ -1,7 +1,6 @@ | ||
1 | 1 | #ifndef OPTIONDLG_H |
2 | 2 | #define OPTIONDLG_H |
3 | 3 | |
4 | -#include <list> | |
5 | 4 | #include <windows.h> |
6 | 5 | |
7 | 6 | #include "../../Common/tstring.h" |
@@ -17,7 +17,7 @@ | ||
17 | 17 | class StrRscMain : public Common::NsmPluginMain |
18 | 18 | { |
19 | 19 | private: |
20 | - typedef std::tr1::unordered_map<std::tstring,std::tstring> IniFilesTable; | |
20 | + typedef std::tr1::unordered_map<std::tstring, std::tstring> IniFilesTable; | |
21 | 21 | |
22 | 22 | IniFilesTable IniFiles; |
23 | 23 | HNsmEvent ChangeLangEvent; |
@@ -24,15 +24,15 @@ | ||
24 | 24 | |
25 | 25 | StrRscConfig Config; |
26 | 26 | //-- |
27 | - static int StrToStrRsc(const PNsmStrRsc,const std::tstring &source); | |
28 | - void DoInitialize(); | |
27 | + static int StrToStrRsc(const NsmStrRsc &strRsc, const std::tstring &source); | |
28 | + void DoInitialize(); | |
29 | 29 | void RegisterIniFile(const std::tstring &pluginName); |
30 | 30 | public: |
31 | 31 | StrRscMain(); |
32 | - int DoGetLocalStr(const PNsmStrRsc strRsc); | |
33 | - INT_PTR DoShowOptionDialog(const HWND parent); | |
32 | + int DoGetLocalStr(const NsmStrRsc &strRsc); | |
33 | + INT_PTR DoShowOptionDialog(HWND parent); | |
34 | 34 | }; |
35 | - | |
35 | + | |
36 | 36 | // サービス関数 |
37 | 37 | class StrRscService |
38 | 38 | { |
@@ -47,7 +47,7 @@ | ||
47 | 47 | |
48 | 48 | extern "C" |
49 | 49 | { |
50 | - int WINAPI GetPluginInfo(const int nInfNo, LPSTR lpBuffer,const int nSize); | |
50 | + int WINAPI GetPluginInfo(const int nInfNo, LPSTR lpBuffer, const int nSize); | |
51 | 51 | int WINAPI Initialize(const Regnessem::PNsmPluginInitInfo lpNsmInitInfo); |
52 | 52 | int WINAPI Terminate(); |
53 | 53 | } |
@@ -43,7 +43,7 @@ | ||
43 | 43 | StrRsc::g_StrResource.reset(new StrRsc::StrResource(InitInfo)); |
44 | 44 | { |
45 | 45 | NsmThreadClientSocketList::AutoLock lock(SocketList); |
46 | - lock.List()->clear(); | |
46 | + lock.List().clear(); | |
47 | 47 | } |
48 | 48 | Config.LoadFromFile(GetIniFileName()); |
49 | 49 |
@@ -65,7 +65,7 @@ | ||
65 | 65 | void ClientSocketMain::DoTerminate() |
66 | 66 | { |
67 | 67 | NsmThreadClientSocketList::AutoLock lock(SocketList); |
68 | - lock.List()->clear(); | |
68 | + lock.List().clear(); | |
69 | 69 | } |
70 | 70 | |
71 | 71 | // ソケットを作成 |
@@ -80,7 +80,7 @@ | ||
80 | 80 | |
81 | 81 | { |
82 | 82 | NsmThreadClientSocketList::AutoLock lock(SocketList); |
83 | - lock.List()->push_back(NsmClientSocketListSub(soc)); | |
83 | + lock.List().push_back(NsmClientSocketListSub(soc)); | |
84 | 84 | } |
85 | 85 | |
86 | 86 | return reinterpret_cast<LONG_PTR>(soc); |
@@ -93,7 +93,7 @@ | ||
93 | 93 | |
94 | 94 | { |
95 | 95 | NsmThreadClientSocketList::AutoLock lock(SocketList); |
96 | - NsmClientSocketList &p = *lock.List(); | |
96 | + NsmClientSocketList &p = lock.List(); | |
97 | 97 | |
98 | 98 | NsmClientSocketList::const_iterator it = |
99 | 99 | std::find_if(p.begin(),p.end(),NsmClientSocket::FindByHandle(handle)); |
@@ -109,13 +109,13 @@ | ||
109 | 109 | } |
110 | 110 | |
111 | 111 | // ソケットを開く |
112 | - LONG_PTR ClientSocketMain::DoOpen(NsmClientSocket *const handle) | |
112 | + LONG_PTR ClientSocketMain::DoOpen(NsmClientSocket *const handle) const | |
113 | 113 | { |
114 | 114 | RgnOutputDebugString(TEXT("CClientSocketMain::DoOpen()\n")); |
115 | 115 | |
116 | 116 | { |
117 | - NsmThreadClientSocketList::AutoLock lock(SocketList); | |
118 | - const NsmClientSocketList &p = *lock.List(); | |
117 | + NsmThreadClientSocketList::ConstAutoLock lock(SocketList); | |
118 | + const NsmClientSocketList &p = lock.List(); | |
119 | 119 | |
120 | 120 | if(std::find_if(p.begin(),p.end(),NsmClientSocket::FindByHandle(handle)) == p.end()) |
121 | 121 | return 0; |
@@ -132,13 +132,13 @@ | ||
132 | 132 | } |
133 | 133 | |
134 | 134 | // ソケットを閉じる |
135 | - LONG_PTR ClientSocketMain::DoClose(NsmClientSocket *const handle) | |
135 | + LONG_PTR ClientSocketMain::DoClose(NsmClientSocket *const handle) const | |
136 | 136 | { |
137 | 137 | RgnOutputDebugString(TEXT("CClientSocketMain::DoClose()\n")); |
138 | 138 | |
139 | 139 | { |
140 | - NsmThreadClientSocketList::AutoLock lock(SocketList); | |
141 | - const NsmClientSocketList &p = *lock.List(); | |
140 | + NsmThreadClientSocketList::ConstAutoLock lock(SocketList); | |
141 | + const NsmClientSocketList &p = lock.List(); | |
142 | 142 | |
143 | 143 | if(std::find_if(p.begin(),p.end(),NsmClientSocket::FindByHandle(handle)) == p.end()) |
144 | 144 | return 0; |
@@ -148,7 +148,7 @@ | ||
148 | 148 | } |
149 | 149 | |
150 | 150 | // ソケットの情報を取得 |
151 | - LONG_PTR ClientSocketMain::DoGetInfo(const NsmClientSocket *const handle, const PNsmClientSocketInfo sockInfo) | |
151 | + LONG_PTR ClientSocketMain::DoGetInfo(const NsmClientSocket *const handle, const PNsmClientSocketInfo sockInfo) const | |
152 | 152 | { |
153 | 153 | RgnOutputDebugString(TEXT("CClientSocketMain::DoGetInfo()\n")); |
154 | 154 |
@@ -156,8 +156,8 @@ | ||
156 | 156 | return 0; |
157 | 157 | |
158 | 158 | { |
159 | - NsmThreadClientSocketList::AutoLock lock(SocketList); | |
160 | - const NsmClientSocketList &p = *lock.List(); | |
159 | + NsmThreadClientSocketList::ConstAutoLock lock(SocketList); | |
160 | + const NsmClientSocketList &p = lock.List(); | |
161 | 161 | |
162 | 162 | if(std::find_if(p.begin(),p.end(),NsmClientSocket::FindByHandle(handle)) == p.end()) |
163 | 163 | return 0; |
@@ -206,7 +206,7 @@ | ||
206 | 206 | } |
207 | 207 | |
208 | 208 | // ソケットの情報を設定 |
209 | - LONG_PTR ClientSocketMain::DoSetInfo(NsmClientSocket *const handle, const NsmClientSocketInfo &sockInfo) | |
209 | + LONG_PTR ClientSocketMain::DoSetInfo(NsmClientSocket *const handle, const NsmClientSocketInfo &sockInfo) const | |
210 | 210 | { |
211 | 211 | RgnOutputDebugString(TEXT("CClientSocketMain::DoSetInfo()\n")); |
212 | 212 |
@@ -214,8 +214,8 @@ | ||
214 | 214 | return 0; |
215 | 215 | |
216 | 216 | { |
217 | - NsmThreadClientSocketList::AutoLock lock(SocketList); | |
218 | - const NsmClientSocketList &p = *lock.List(); | |
217 | + NsmThreadClientSocketList::ConstAutoLock lock(SocketList); | |
218 | + const NsmClientSocketList &p = lock.List(); | |
219 | 219 | |
220 | 220 | if(std::find_if(p.begin(),p.end(),NsmClientSocket::FindByHandle(handle)) == p.end()) |
221 | 221 | return 0; |
@@ -236,7 +236,7 @@ | ||
236 | 236 | return 0; |
237 | 237 | } |
238 | 238 | |
239 | - LONG_PTR ClientSocketMain::DoReceive(const NsmClientSocket *const handle, const NsmClientSocketData &data) | |
239 | + LONG_PTR ClientSocketMain::DoReceive(const NsmClientSocket *const handle, const NsmClientSocketData &data) const | |
240 | 240 | { |
241 | 241 | RgnOutputDebugString(TEXT("CClientSocketMain::DoReceive()\n")); |
242 | 242 |
@@ -244,8 +244,8 @@ | ||
244 | 244 | return 0; |
245 | 245 | |
246 | 246 | { |
247 | - NsmThreadClientSocketList::AutoLock lock(SocketList); | |
248 | - const NsmClientSocketList &p = *lock.List(); | |
247 | + NsmThreadClientSocketList::ConstAutoLock lock(SocketList); | |
248 | + const NsmClientSocketList &p = lock.List(); | |
249 | 249 | |
250 | 250 | if(std::find_if(p.begin(),p.end(),NsmClientSocket::FindByHandle(handle)) == p.end()) |
251 | 251 | return 0; |
@@ -254,7 +254,7 @@ | ||
254 | 254 | } |
255 | 255 | } |
256 | 256 | |
257 | - LONG_PTR ClientSocketMain::DoSend(const NsmClientSocket *const handle, const NsmClientSocketData &data) | |
257 | + LONG_PTR ClientSocketMain::DoSend(const NsmClientSocket *const handle, const NsmClientSocketData &data) const | |
258 | 258 | { |
259 | 259 | RgnOutputDebugString(TEXT("CClientSocketMain::DoSend()\n")); |
260 | 260 |
@@ -262,8 +262,8 @@ | ||
262 | 262 | return 0; |
263 | 263 | |
264 | 264 | { |
265 | - NsmThreadClientSocketList::AutoLock lock(SocketList); | |
266 | - const NsmClientSocketList &p = *lock.List(); | |
265 | + NsmThreadClientSocketList::ConstAutoLock lock(SocketList); | |
266 | + const NsmClientSocketList &p = lock.List(); | |
267 | 267 | |
268 | 268 | if(std::find_if(p.begin(),p.end(),NsmClientSocket::FindByHandle(handle)) == p.end()) |
269 | 269 | return 0; |
@@ -26,12 +26,12 @@ | ||
26 | 26 | // サービス処理関数 |
27 | 27 | LONG_PTR DoCreate(const NsmClientSocketInitInfo &initInfo); |
28 | 28 | LONG_PTR DoDelete(const NsmClientSocket *const handle); |
29 | - LONG_PTR DoOpen(NsmClientSocket *const handle); | |
30 | - LONG_PTR DoClose(NsmClientSocket *const handle); | |
31 | - LONG_PTR DoGetInfo(const NsmClientSocket *const handle, const PNsmClientSocketInfo sockInfo); | |
32 | - LONG_PTR DoSetInfo(NsmClientSocket *const handle, const NsmClientSocketInfo &sockInfo); | |
33 | - LONG_PTR DoReceive(const NsmClientSocket *const handle, const NsmClientSocketData &data); | |
34 | - LONG_PTR DoSend(const NsmClientSocket *const handle, const NsmClientSocketData &data); | |
29 | + LONG_PTR DoOpen(NsmClientSocket *const handle) const; | |
30 | + LONG_PTR DoClose(NsmClientSocket *const handle) const; | |
31 | + LONG_PTR DoGetInfo(const NsmClientSocket *const handle, const PNsmClientSocketInfo sockInfo) const; | |
32 | + LONG_PTR DoSetInfo(NsmClientSocket *const handle, const NsmClientSocketInfo &sockInfo) const; | |
33 | + LONG_PTR DoReceive(const NsmClientSocket *const handle, const NsmClientSocketData &data) const; | |
34 | + LONG_PTR DoSend(const NsmClientSocket *const handle, const NsmClientSocketData &data) const; | |
35 | 35 | |
36 | 36 | LONG_PTR DoShowOptionDialog(const HWND parent); |
37 | 37 |
@@ -194,7 +194,7 @@ | ||
194 | 194 | return 0; |
195 | 195 | |
196 | 196 | NsmThreadFileIOList::AutoLock lock(Files); |
197 | - NsmFileIOList &p = *lock.List(); | |
197 | + NsmFileIOList &p = lock.List(); | |
198 | 198 | |
199 | 199 | NsmFileIO* newIO = new NsmFileIO(); |
200 | 200 |
@@ -213,7 +213,7 @@ | ||
213 | 213 | + ((*Config.FileDir.rbegin() == TEXT('\\'))?(filename):(TEXT("\\") + filename)); |
214 | 214 | |
215 | 215 | NsmThreadFileIOList::AutoLock lock(Files); |
216 | - NsmFileIOList &p = *lock.List(); | |
216 | + NsmFileIOList &p = lock.List(); | |
217 | 217 | |
218 | 218 | NsmFileIO* newIO = new NsmFileIO(); |
219 | 219 |
@@ -222,10 +222,10 @@ | ||
222 | 222 | return newIO->OpenNew(newPath) ? reinterpret_cast<LONG_PTR>(newIO) : 0; |
223 | 223 | } |
224 | 224 | |
225 | - LONG_PTR FileIOMain::DoReadFile(const NsmFileIO *const fileIO, const PNsmFileIOData data) | |
225 | + LONG_PTR FileIOMain::DoReadFile(const NsmFileIO *const fileIO, const PNsmFileIOData data) const | |
226 | 226 | { |
227 | - NsmThreadFileIOList::AutoLock lock(Files); | |
228 | - const NsmFileIOList &p = *lock.List(); | |
227 | + NsmThreadFileIOList::ConstAutoLock lock(Files); | |
228 | + const NsmFileIOList &p = lock.List(); | |
229 | 229 | |
230 | 230 | if(std::find_if(p.begin(),p.end(),NsmFileIO::FindByHandle(fileIO)) == p.end()) |
231 | 231 | return 0; |
@@ -233,10 +233,10 @@ | ||
233 | 233 | return fileIO->ReadData(data); |
234 | 234 | } |
235 | 235 | |
236 | - LONG_PTR FileIOMain::DoAppendFile(const NsmFileIO *const fileIO, const NsmFileIOData &data) | |
236 | + LONG_PTR FileIOMain::DoAppendFile(const NsmFileIO *const fileIO, const NsmFileIOData &data) const | |
237 | 237 | { |
238 | - NsmThreadFileIOList::AutoLock lock(Files); | |
239 | - const NsmFileIOList &p = *lock.List(); | |
238 | + NsmThreadFileIOList::ConstAutoLock lock(Files); | |
239 | + const NsmFileIOList &p = lock.List(); | |
240 | 240 | |
241 | 241 | if(std::find_if(p.begin(),p.end(),NsmFileIO::FindByHandle(fileIO)) == p.end()) |
242 | 242 | return 0; |
@@ -247,7 +247,7 @@ | ||
247 | 247 | LONG_PTR FileIOMain::DoCloseFile(NsmFileIO *const fileIO) |
248 | 248 | { |
249 | 249 | NsmThreadFileIOList::AutoLock lock(Files); |
250 | - NsmFileIOList &p = *lock.List(); | |
250 | + NsmFileIOList &p = lock.List(); | |
251 | 251 | |
252 | 252 | NsmFileIOList::iterator it = std::find_if(p.begin(),p.end(),NsmFileIO::FindByHandle(fileIO)); |
253 | 253 |
@@ -261,10 +261,10 @@ | ||
261 | 261 | return result; |
262 | 262 | } |
263 | 263 | |
264 | - LONG_PTR FileIOMain::DoFileGetInfo(const NsmFileIO *const fileIO, const PNsmFileIOInfo fInfo) | |
264 | + LONG_PTR FileIOMain::DoFileGetInfo(const NsmFileIO *const fileIO, const PNsmFileIOInfo fInfo) const | |
265 | 265 | { |
266 | - NsmThreadFileIOList::AutoLock lock(Files); | |
267 | - const NsmFileIOList &p = *lock.List(); | |
266 | + NsmThreadFileIOList::ConstAutoLock lock(Files); | |
267 | + const NsmFileIOList &p = lock.List(); | |
268 | 268 | |
269 | 269 | if(std::find_if(p.begin(),p.end(),NsmFileIO::FindByHandle(fileIO)) == p.end()) |
270 | 270 | return 0; |
@@ -64,10 +64,10 @@ | ||
64 | 64 | FileIOMain(); |
65 | 65 | LONG_PTR DoOpenFile(const std::tstring &filePath); |
66 | 66 | LONG_PTR DoCreateFile(const std::tstring &filename); |
67 | - LONG_PTR DoReadFile(const NsmFileIO *const fileIO, const PNsmFileIOData data); | |
68 | - LONG_PTR DoAppendFile(const NsmFileIO *const fileIO, const NsmFileIOData &data); | |
67 | + LONG_PTR DoReadFile(const NsmFileIO *const fileIO, const PNsmFileIOData data) const; | |
68 | + LONG_PTR DoAppendFile(const NsmFileIO *const fileIO, const NsmFileIOData &data) const; | |
69 | 69 | LONG_PTR DoCloseFile(NsmFileIO *const fileIO); |
70 | - LONG_PTR DoFileGetInfo(const NsmFileIO *const fileIO, const PNsmFileIOInfo fInfo); | |
70 | + LONG_PTR DoFileGetInfo(const NsmFileIO *const fileIO, const PNsmFileIOInfo fInfo) const; | |
71 | 71 | |
72 | 72 | LONG_PTR UIOpenFileDir(const PNsmUIServiceInfo uiInfo) const; |
73 | 73 | LONG_PTR DoShowOptionDialog(const HWND parent); |
@@ -48,7 +48,7 @@ | ||
48 | 48 | |
49 | 49 | std::tstring NsmPluginMain::GetPluginInfo(const int index) const |
50 | 50 | { |
51 | - if(NMPI_APIVER <= index && index <= NMPI_PLUGINVER) | |
51 | + if(0 <= index && index < ARRAYSIZE(PluginInfo)) | |
52 | 52 | return PluginInfo[index]; |
53 | 53 | |
54 | 54 | return TEXT(""); |
@@ -56,7 +56,7 @@ | ||
56 | 56 | |
57 | 57 | void NsmPluginMain::SetPluginInfo(const int index,const std::tstring &value) |
58 | 58 | { |
59 | - if(NMPI_APIVER <= index && index <= NMPI_PLUGINVER) | |
59 | + if(0 <= index && index < ARRAYSIZE(PluginInfo)) | |
60 | 60 | PluginInfo[index] = value; |
61 | 61 | } |
62 | 62 |
@@ -47,10 +47,6 @@ | ||
47 | 47 | explicit AutoLock(TThreadList &self) |
48 | 48 | :Self(self), Lock(self.CS) |
49 | 49 | {} |
50 | - ListType* List() | |
51 | - { | |
52 | - return &Self.List; | |
53 | - } | |
54 | 50 | ListType& List() const |
55 | 51 | { |
56 | 52 | return Self.List; |
@@ -57,11 +57,11 @@ | ||
57 | 57 | NsmServiceListSub result; |
58 | 58 | { |
59 | 59 | NsmThreadServiceList::AutoLock lock(ServiceList); |
60 | - NsmServiceList *const p = lock.List(); | |
60 | + NsmServiceList &p = lock.List(); | |
61 | 61 | |
62 | - result.reset(new NsmService(p,procAddr)); | |
62 | + result.reset(new NsmService(&p,procAddr)); | |
63 | 63 | |
64 | - if(!p->insert(NsmServiceList::value_type(serviceName,result)).second) | |
64 | + if(!p.insert(NsmServiceList::value_type(serviceName,result)).second) | |
65 | 65 | return 0; |
66 | 66 | } |
67 | 67 |
@@ -69,14 +69,14 @@ | ||
69 | 69 | return reinterpret_cast<HNsmService>(result.get()); |
70 | 70 | } |
71 | 71 | |
72 | - HNsmService TNsmCom::GetService(const std::tstring &serviceName) | |
72 | + HNsmService TNsmCom::GetService(const std::tstring &serviceName) const | |
73 | 73 | { |
74 | 74 | if (DebugMode){ |
75 | 75 | DoLog(format(TEXT("GetService\t\"%s\""), serviceName.c_str())); |
76 | 76 | } |
77 | 77 | |
78 | - NsmThreadServiceList::AutoLock lock(ServiceList); | |
79 | - const NsmServiceList &p = *lock.List(); | |
78 | + NsmThreadServiceList::ConstAutoLock lock(ServiceList); | |
79 | + const NsmServiceList &p = lock.List(); | |
80 | 80 | |
81 | 81 | NsmServiceList::const_iterator svc = p.find(serviceName); |
82 | 82 |
@@ -107,11 +107,11 @@ | ||
107 | 107 | NsmEventListSub result; |
108 | 108 | { |
109 | 109 | NsmThreadEventList::AutoLock lock(EventList); |
110 | - NsmEventList* p = lock.List(); | |
110 | + NsmEventList &p = lock.List(); | |
111 | 111 | |
112 | - result.reset(new NsmEvent(p)); | |
112 | + result.reset(new NsmEvent(&p)); | |
113 | 113 | |
114 | - if(!p->insert(NsmEventList::value_type(eventName,result)).second) | |
114 | + if(!p.insert(NsmEventList::value_type(eventName,result)).second) | |
115 | 115 | return 0; |
116 | 116 | } |
117 | 117 |
@@ -119,14 +119,14 @@ | ||
119 | 119 | return reinterpret_cast<HNsmEvent>(result.get()); |
120 | 120 | } |
121 | 121 | |
122 | - HNsmEvent TNsmCom::GetEvent(const std::tstring &eventName) | |
122 | + HNsmEvent TNsmCom::GetEvent(const std::tstring &eventName) const | |
123 | 123 | { |
124 | 124 | if (DebugMode){ |
125 | 125 | DoLog(format(TEXT("GetEvent\t\"%s\""), eventName.c_str())); |
126 | 126 | } |
127 | 127 | |
128 | - NsmThreadEventList::AutoLock lock(EventList); | |
129 | - const NsmEventList &p = *lock.List(); | |
128 | + NsmThreadEventList::ConstAutoLock lock(EventList); | |
129 | + const NsmEventList &p = lock.List(); | |
130 | 130 | |
131 | 131 | NsmEventList::const_iterator evt = p.find(eventName); |
132 | 132 |
@@ -163,7 +163,7 @@ | ||
163 | 163 | return 0; |
164 | 164 | } |
165 | 165 | |
166 | - int TNsmCom::HookEvent(const std::tstring &eventName, TNsmEventProc procAddr) | |
166 | + int TNsmCom::HookEvent(const std::tstring &eventName, TNsmEventProc procAddr) const | |
167 | 167 | { |
168 | 168 | if(!procAddr) |
169 | 169 | return 0; |
@@ -173,8 +173,8 @@ | ||
173 | 173 | } |
174 | 174 | |
175 | 175 | { |
176 | - NsmThreadEventList::AutoLock lock(EventList); | |
177 | - const NsmEventList &p = *lock.List(); | |
176 | + NsmThreadEventList::ConstAutoLock lock(EventList); | |
177 | + const NsmEventList &p = lock.List(); | |
178 | 178 | |
179 | 179 | NsmEventList::const_iterator evt = p.find(eventName); |
180 | 180 | if(evt == p.end()) |
@@ -188,7 +188,7 @@ | ||
188 | 188 | return 1; |
189 | 189 | } |
190 | 190 | |
191 | - int TNsmCom::UnhookEvent(const std::tstring &eventName, TNsmEventProc procAddr) | |
191 | + int TNsmCom::UnhookEvent(const std::tstring &eventName, TNsmEventProc procAddr) const | |
192 | 192 | { |
193 | 193 | if(!procAddr) |
194 | 194 | return 0; |
@@ -198,8 +198,8 @@ | ||
198 | 198 | } |
199 | 199 | |
200 | 200 | { |
201 | - NsmThreadEventList::AutoLock lock(EventList); | |
202 | - const NsmEventList &p = *lock.List(); | |
201 | + NsmThreadEventList::ConstAutoLock lock(EventList); | |
202 | + const NsmEventList &p = lock.List(); | |
203 | 203 | |
204 | 204 | NsmEventList::const_iterator evt = p.find(eventName); |
205 | 205 | if(evt == p.end()) |
@@ -156,7 +156,7 @@ | ||
156 | 156 | void NsmSystem::CloseRelatedSessions(const NsmConnection *const connection) |
157 | 157 | { |
158 | 158 | NsmThreadSessionList::AutoLock lock(Sessions); |
159 | - const NsmSessionList &sesList = *lock.List(); | |
159 | + NsmSessionList &sesList = lock.List(); | |
160 | 160 | |
161 | 161 | for(NsmSessionList::size_type i = sesList.size() ; i ; --i) |
162 | 162 | { |
@@ -174,7 +174,7 @@ | ||
174 | 174 | void NsmSystem::CloseAllSession() |
175 | 175 | { |
176 | 176 | NsmThreadSessionList::AutoLock lock(Sessions); |
177 | - const NsmSessionList &sesList = *lock.List(); | |
177 | + NsmSessionList &sesList = lock.List(); | |
178 | 178 | |
179 | 179 | for(NsmSessionList::size_type i = sesList.size() ; i ; --i) |
180 | 180 | { |
@@ -189,7 +189,7 @@ | ||
189 | 189 | void NsmSystem::CloseRelatedFileSessions(const NsmSession *const session) |
190 | 190 | { |
191 | 191 | NsmThreadFileSessionList::AutoLock lock(FileSessions); |
192 | - const NsmFileSessionList &fileSesList = *lock.List(); | |
192 | + NsmFileSessionList &fileSesList = lock.List(); | |
193 | 193 | |
194 | 194 | for(NsmFileSessionList::size_type i = fileSesList.size(); i ; --i) |
195 | 195 | { |
@@ -207,7 +207,7 @@ | ||
207 | 207 | void NsmSystem::CloseAllFileSession() |
208 | 208 | { |
209 | 209 | NsmThreadFileSessionList::AutoLock lock(FileSessions); |
210 | - const NsmFileSessionList &fileSesList = *lock.List(); | |
210 | + NsmFileSessionList &fileSesList = lock.List(); | |
211 | 211 | |
212 | 212 | for(NsmFileSessionList::size_type i = fileSesList.size(); i ; --i) |
213 | 213 | { |
@@ -222,7 +222,7 @@ | ||
222 | 222 | void NsmSystem::DisconnectAll() |
223 | 223 | { |
224 | 224 | NsmThreadConnectionList::AutoLock lock(Connections); |
225 | - const NsmConnectionList &conList = *lock.List(); | |
225 | + NsmConnectionList &conList = lock.List(); | |
226 | 226 | |
227 | 227 | for(NsmConnectionList::size_type i = conList.size(); i ; --i) |
228 | 228 | { |
@@ -438,11 +438,11 @@ | ||
438 | 438 | NsmConnection *result = NULL; |
439 | 439 | { |
440 | 440 | NsmThreadConnectionList::AutoLock lock(g_NsmSystem.Connections); |
441 | - NsmConnectionList *const p = lock.List(); | |
441 | + NsmConnectionList &p = lock.List(); | |
442 | 442 | |
443 | 443 | result = new NsmConnection(c2t(reinterpret_cast<char *>(wParam)),c2t(reinterpret_cast<char *>(lParam == 0?wParam:lParam))); |
444 | 444 | |
445 | - p->push_back(NsmConnectionListSub(result)); | |
445 | + p.push_back(NsmConnectionListSub(result)); | |
446 | 446 | } |
447 | 447 | |
448 | 448 | NsmCom.NotifyEvent(g_NsmSystem.HConnectionConnect,reinterpret_cast<WPARAM>(result), 0); |
@@ -455,15 +455,15 @@ | ||
455 | 455 | { |
456 | 456 | { |
457 | 457 | NsmThreadConnectionList::AutoLock lock(g_NsmSystem.Connections); |
458 | - NsmConnectionList *const p = lock.List(); | |
458 | + NsmConnectionList &p = lock.List(); | |
459 | 459 | |
460 | 460 | NsmConnectionList::iterator it = |
461 | - std::find_if(p->begin(), p->end(),NsmConnection::FindByHandle(reinterpret_cast<NsmConnection *>(wParam))); | |
462 | - if(it == p->end()) | |
461 | + std::find_if(p.begin(), p.end(),NsmConnection::FindByHandle(reinterpret_cast<NsmConnection *>(wParam))); | |
462 | + if(it == p.end()) | |
463 | 463 | return -1; |
464 | 464 | |
465 | 465 | g_NsmSystem.CloseRelatedSessions(reinterpret_cast<NsmConnection *>(wParam)); |
466 | - p->erase(it); | |
466 | + p.erase(it); | |
467 | 467 | } |
468 | 468 | |
469 | 469 | NsmCom.NotifyEvent(g_NsmSystem.HConnectionDisconnect, wParam, 0); |
@@ -477,8 +477,8 @@ | ||
477 | 477 | const NsmConnection *const connection = reinterpret_cast<NsmConnection *>(wParam); |
478 | 478 | const PNsmConnectionInfo pin = reinterpret_cast<PNsmConnectionInfo>(lParam); |
479 | 479 | |
480 | - NsmThreadConnectionList::AutoLock lock(g_NsmSystem.Connections); | |
481 | - const NsmConnectionList &p = *lock.List(); | |
480 | + NsmThreadConnectionList::ConstAutoLock lock(g_NsmSystem.Connections); | |
481 | + const NsmConnectionList &p = lock.List(); | |
482 | 482 | |
483 | 483 | if (std::find_if(p.begin(), p.end(), NsmConnection::FindByHandle(connection)) == p.end()) |
484 | 484 | return 0; |
@@ -524,8 +524,8 @@ | ||
524 | 524 | { |
525 | 525 | NsmConnection *const connection = reinterpret_cast<NsmConnection *>(wParam); |
526 | 526 | |
527 | - NsmThreadConnectionList::AutoLock lock(g_NsmSystem.Connections); | |
528 | - const NsmConnectionList &p = *lock.List(); | |
527 | + NsmThreadConnectionList::ConstAutoLock lock(g_NsmSystem.Connections); | |
528 | + const NsmConnectionList &p = lock.List(); | |
529 | 529 | |
530 | 530 | if (std::find_if(p.begin(), p.end(), NsmConnection::FindByHandle(connection)) == p.end()) |
531 | 531 | return 0; |
@@ -586,8 +586,8 @@ | ||
586 | 586 | |
587 | 587 | LONG_PTR Service::DoEnumConnections(const WPARAM wParam,const LPARAM lParam) |
588 | 588 | { |
589 | - NsmThreadConnectionList::AutoLock lock(g_NsmSystem.Connections); | |
590 | - const NsmConnectionList &p = *lock.List(); | |
589 | + NsmThreadConnectionList::ConstAutoLock lock(g_NsmSystem.Connections); | |
590 | + const NsmConnectionList &p = lock.List(); | |
591 | 591 | |
592 | 592 | for (NsmConnectionList::const_iterator it = p.begin(); it != p.end(); ++it) |
593 | 593 | reinterpret_cast<TEnumConnectionCallback>(wParam) (reinterpret_cast<HNsmConnection>(it->get()), lParam); |
@@ -600,8 +600,8 @@ | ||
600 | 600 | LONG_PTR Service::DoAddMember(const WPARAM wParam,const LPARAM lParam) |
601 | 601 | { |
602 | 602 | { |
603 | - NsmThreadConnectionList::AutoLock lock(g_NsmSystem.Connections); | |
604 | - const NsmConnectionList &p = *lock.List(); | |
603 | + NsmThreadConnectionList::ConstAutoLock lock(g_NsmSystem.Connections); | |
604 | + const NsmConnectionList &p = lock.List(); | |
605 | 605 | NsmConnectionList::const_iterator it = |
606 | 606 | std::find_if(p.begin(), p.end(), NsmConnection::FindByHandle(reinterpret_cast<NsmConnection *>(wParam))); |
607 | 607 | if (it == p.end()) |
@@ -624,8 +624,8 @@ | ||
624 | 624 | LONG_PTR Service::DoRemoveMember(const WPARAM wParam,const LPARAM lParam) |
625 | 625 | { |
626 | 626 | { |
627 | - NsmThreadConnectionList::AutoLock lock(g_NsmSystem.Connections); | |
628 | - const NsmConnectionList &p = *lock.List(); | |
627 | + NsmThreadConnectionList::ConstAutoLock lock(g_NsmSystem.Connections); | |
628 | + const NsmConnectionList &p = lock.List(); | |
629 | 629 | NsmConnectionList::const_iterator it = |
630 | 630 | std::find_if(p.begin(), p.end(), NsmConnection::FindByHandle(reinterpret_cast<NsmConnection *>(wParam))); |
631 | 631 | if (it == p.end()) |
@@ -646,8 +646,8 @@ | ||
646 | 646 | { |
647 | 647 | const TNsmMemberInfo &MemberInfo = *reinterpret_cast<PNsmMemberInfo>(lParam); |
648 | 648 | { |
649 | - NsmThreadConnectionList::AutoLock lock(g_NsmSystem.Connections); | |
650 | - const NsmConnectionList &p = *lock.List(); | |
649 | + NsmThreadConnectionList::ConstAutoLock lock(g_NsmSystem.Connections); | |
650 | + const NsmConnectionList &p = lock.List(); | |
651 | 651 | NsmConnectionList::const_iterator it = |
652 | 652 | std::find_if(p.begin(), p.end(), NsmConnection::FindByHandle(reinterpret_cast<NsmConnection *>(wParam))); |
653 | 653 | if (it == p.end()) |
@@ -704,8 +704,8 @@ | ||
704 | 704 | |
705 | 705 | LONG_PTR Service::DoGetMemberInfo(const WPARAM wParam,const LPARAM lParam) |
706 | 706 | { |
707 | - NsmThreadConnectionList::AutoLock lock(g_NsmSystem.Connections); | |
708 | - const NsmConnectionList &p = *lock.List(); | |
707 | + NsmThreadConnectionList::ConstAutoLock lock(g_NsmSystem.Connections); | |
708 | + const NsmConnectionList &p = lock.List(); | |
709 | 709 | NsmConnectionList::const_iterator it = |
710 | 710 | std::find_if(p.begin(), p.end(), NsmConnection::FindByHandle(reinterpret_cast<NsmConnection *>(wParam))); |
711 | 711 | if (it == p.end()) |
@@ -749,8 +749,8 @@ | ||
749 | 749 | |
750 | 750 | LONG_PTR Service::DoEnumMembers(const WPARAM wParam,const LPARAM lParam) |
751 | 751 | { |
752 | - NsmThreadConnectionList::AutoLock lock(g_NsmSystem.Connections); | |
753 | - const NsmConnectionList &p = *lock.List(); | |
752 | + NsmThreadConnectionList::ConstAutoLock lock(g_NsmSystem.Connections); | |
753 | + const NsmConnectionList &p = lock.List(); | |
754 | 754 | NsmConnectionList::const_iterator it = |
755 | 755 | std::find_if(p.begin(), p.end(), NsmConnection::FindByHandle(reinterpret_cast<NsmConnection *>(wParam))); |
756 | 756 | if (it == p.end()) |
@@ -782,8 +782,8 @@ | ||
782 | 782 | LONG_PTR Service::DoAddGroup(const WPARAM wParam,const LPARAM lParam) |
783 | 783 | { |
784 | 784 | { |
785 | - NsmThreadConnectionList::AutoLock lock(g_NsmSystem.Connections); | |
786 | - const NsmConnectionList &p = *lock.List(); | |
785 | + NsmThreadConnectionList::ConstAutoLock lock(g_NsmSystem.Connections); | |
786 | + const NsmConnectionList &p = lock.List(); | |
787 | 787 | NsmConnectionList::const_iterator it = |
788 | 788 | std::find_if(p.begin(), p.end(), NsmConnection::FindByHandle(reinterpret_cast<NsmConnection *>(wParam))); |
789 | 789 | if (it == p.end()) |
@@ -807,8 +807,8 @@ | ||
807 | 807 | LONG_PTR Service::DoRemoveGroup(const WPARAM wParam,const LPARAM lParam) |
808 | 808 | { |
809 | 809 | { |
810 | - NsmThreadConnectionList::AutoLock lock(g_NsmSystem.Connections); | |
811 | - const NsmConnectionList &p = *lock.List(); | |
810 | + NsmThreadConnectionList::ConstAutoLock lock(g_NsmSystem.Connections); | |
811 | + const NsmConnectionList &p = lock.List(); | |
812 | 812 | NsmConnectionList::const_iterator it = |
813 | 813 | std::find_if(p.begin(), p.end(), NsmConnection::FindByHandle(reinterpret_cast<NsmConnection *>(wParam))); |
814 | 814 | if (it == p.end()) |
@@ -830,8 +830,8 @@ | ||
830 | 830 | { |
831 | 831 | TNsmGroupInfo & GroupInfo = *reinterpret_cast<PNsmGroupInfo>(lParam); |
832 | 832 | { |
833 | - NsmThreadConnectionList::AutoLock lock(g_NsmSystem.Connections); | |
834 | - const NsmConnectionList &p = *lock.List(); | |
833 | + NsmThreadConnectionList::ConstAutoLock lock(g_NsmSystem.Connections); | |
834 | + const NsmConnectionList &p = lock.List(); | |
835 | 835 | NsmConnectionList::const_iterator it = |
836 | 836 | std::find_if(p.begin(), p.end(), NsmConnection::FindByHandle(reinterpret_cast<NsmConnection *>(wParam))); |
837 | 837 | if (it == p.end()) |
@@ -865,8 +865,8 @@ | ||
865 | 865 | |
866 | 866 | LONG_PTR Service::DoGetGroupInfo(const WPARAM wParam,const LPARAM lParam) |
867 | 867 | { |
868 | - NsmThreadConnectionList::AutoLock lock(g_NsmSystem.Connections); | |
869 | - const NsmConnectionList &p = *lock.List(); | |
868 | + NsmThreadConnectionList::ConstAutoLock lock(g_NsmSystem.Connections); | |
869 | + const NsmConnectionList &p = lock.List(); | |
870 | 870 | NsmConnectionList::const_iterator it = |
871 | 871 | std::find_if(p.begin(), p.end(), NsmConnection::FindByHandle(reinterpret_cast<NsmConnection *>(wParam))); |
872 | 872 | if (it == p.end()) |
@@ -896,8 +896,8 @@ | ||
896 | 896 | |
897 | 897 | LONG_PTR Service::DoEnumGroups(const WPARAM wParam,const LPARAM lParam) |
898 | 898 | { |
899 | - NsmThreadConnectionList::AutoLock lock(g_NsmSystem.Connections); | |
900 | - const NsmConnectionList &p = *lock.List(); | |
899 | + NsmThreadConnectionList::ConstAutoLock lock(g_NsmSystem.Connections); | |
900 | + const NsmConnectionList &p = lock.List(); | |
901 | 901 | NsmConnectionList::const_iterator it = |
902 | 902 | std::find_if(p.begin(), p.end(), NsmConnection::FindByHandle(reinterpret_cast<NsmConnection *>(wParam))); |
903 | 903 | if (it == p.end()) |
@@ -929,8 +929,8 @@ | ||
929 | 929 | { |
930 | 930 | NsmSession *result = NULL; |
931 | 931 | { |
932 | - NsmThreadConnectionList::AutoLock lock(g_NsmSystem.Connections); | |
933 | - const NsmConnectionList &p = *lock.List(); | |
932 | + NsmThreadConnectionList::ConstAutoLock lock(g_NsmSystem.Connections); | |
933 | + const NsmConnectionList &p = lock.List(); | |
934 | 934 | NsmConnectionList::const_iterator it = |
935 | 935 | std::find_if(p.begin(), p.end(), NsmConnection::FindByHandle(reinterpret_cast<NsmConnection *>(wParam))); |
936 | 936 | if (it == p.end()) |
@@ -937,7 +937,7 @@ | ||
937 | 937 | return 0; |
938 | 938 | |
939 | 939 | NsmThreadSessionList::AutoLock lock2(g_NsmSystem.Sessions); |
940 | - NsmSessionList &p2 = *lock2.List(); | |
940 | + NsmSessionList &p2 = lock2.List(); | |
941 | 941 | |
942 | 942 | result = new NsmSession(); |
943 | 943 |
@@ -958,7 +958,7 @@ | ||
958 | 958 | { |
959 | 959 | { |
960 | 960 | NsmThreadSessionList::AutoLock lock(g_NsmSystem.Sessions); |
961 | - NsmSessionList &p = *lock.List(); | |
961 | + NsmSessionList &p = lock.List(); | |
962 | 962 | |
963 | 963 | NsmSessionList::iterator it = |
964 | 964 | std::find_if(p.begin(), p.end(), NsmSession::FindByHandle(reinterpret_cast<NsmSession *>(wParam))); |
@@ -976,8 +976,8 @@ | ||
976 | 976 | |
977 | 977 | LONG_PTR Service::DoGetSessionInfo(const WPARAM wParam,const LPARAM lParam) |
978 | 978 | { |
979 | - NsmThreadSessionList::AutoLock lock(g_NsmSystem.Sessions); | |
980 | - const NsmSessionList &p = *lock.List(); | |
979 | + NsmThreadSessionList::ConstAutoLock lock(g_NsmSystem.Sessions); | |
980 | + const NsmSessionList &p = lock.List(); | |
981 | 981 | |
982 | 982 | NsmSessionList::const_iterator it = |
983 | 983 | std::find_if(p.begin(), p.end(), NsmSession::FindByHandle(reinterpret_cast<NsmSession *>(wParam))); |
@@ -1008,8 +1008,8 @@ | ||
1008 | 1008 | LONG_PTR Service::DoSetSessionInfo(const WPARAM wParam,const LPARAM lParam) |
1009 | 1009 | { |
1010 | 1010 | { |
1011 | - NsmThreadSessionList::AutoLock lock(g_NsmSystem.Sessions); | |
1012 | - const NsmSessionList &p = *lock.List(); | |
1011 | + NsmThreadSessionList::ConstAutoLock lock(g_NsmSystem.Sessions); | |
1012 | + const NsmSessionList &p = lock.List(); | |
1013 | 1013 | |
1014 | 1014 | NsmSessionList::const_iterator it = |
1015 | 1015 | std::find_if(p.begin(), p.end(), NsmSession::FindByHandle(reinterpret_cast<NsmSession *>(wParam))); |
@@ -1047,8 +1047,8 @@ | ||
1047 | 1047 | |
1048 | 1048 | LONG_PTR Service::DoEnumSessions(const WPARAM wParam,const LPARAM lParam) |
1049 | 1049 | { |
1050 | - NsmThreadSessionList::AutoLock lock(g_NsmSystem.Sessions); | |
1051 | - const NsmSessionList &p = *lock.List(); | |
1050 | + NsmThreadSessionList::ConstAutoLock lock(g_NsmSystem.Sessions); | |
1051 | + const NsmSessionList &p = lock.List(); | |
1052 | 1052 | |
1053 | 1053 | for(NsmSessionList::const_iterator it = p.begin();it!=p.end();++it) |
1054 | 1054 | reinterpret_cast<TEnumSessionCallback>(wParam)(reinterpret_cast<HNsmSession>(it->get()), lParam); |
@@ -1059,8 +1059,8 @@ | ||
1059 | 1059 | LONG_PTR Service::DoAddSessionMember(const WPARAM wParam,const LPARAM lParam) |
1060 | 1060 | { |
1061 | 1061 | { |
1062 | - NsmThreadSessionList::AutoLock lock(g_NsmSystem.Sessions); | |
1063 | - const NsmSessionList &p = *lock.List(); | |
1062 | + NsmThreadSessionList::ConstAutoLock lock(g_NsmSystem.Sessions); | |
1063 | + const NsmSessionList &p = lock.List(); | |
1064 | 1064 | |
1065 | 1065 | NsmSessionList::const_iterator it = |
1066 | 1066 | std::find_if(p.begin(), p.end(), NsmSession::FindByHandle(reinterpret_cast<NsmSession *>(wParam))); |
@@ -1083,8 +1083,8 @@ | ||
1083 | 1083 | LONG_PTR Service::DoRemoveSessionMember(const WPARAM wParam,const LPARAM lParam) |
1084 | 1084 | { |
1085 | 1085 | { |
1086 | - NsmThreadSessionList::AutoLock lock(g_NsmSystem.Sessions); | |
1087 | - const NsmSessionList &p = *lock.List(); | |
1086 | + NsmThreadSessionList::ConstAutoLock lock(g_NsmSystem.Sessions); | |
1087 | + const NsmSessionList &p = lock.List(); | |
1088 | 1088 | |
1089 | 1089 | NsmSessionList::const_iterator it = |
1090 | 1090 | std::find_if(p.begin(), p.end(), NsmSession::FindByHandle(reinterpret_cast<NsmSession *>(wParam))); |
@@ -1105,8 +1105,8 @@ | ||
1105 | 1105 | LONG_PTR Service::DoSetSessionMemberInfo(const WPARAM wParam,const LPARAM lParam) |
1106 | 1106 | { |
1107 | 1107 | { |
1108 | - NsmThreadSessionList::AutoLock lock(g_NsmSystem.Sessions); | |
1109 | - const NsmSessionList &p = *lock.List(); | |
1108 | + NsmThreadSessionList::ConstAutoLock lock(g_NsmSystem.Sessions); | |
1109 | + const NsmSessionList &p = lock.List(); | |
1110 | 1110 | |
1111 | 1111 | NsmSessionList::const_iterator it = |
1112 | 1112 | std::find_if(p.begin(), p.end(), NsmSession::FindByHandle(reinterpret_cast<NsmSession *>(wParam))); |
@@ -1149,8 +1149,8 @@ | ||
1149 | 1149 | |
1150 | 1150 | LONG_PTR Service::DoGetSessionMemberInfo(const WPARAM wParam,const LPARAM lParam) |
1151 | 1151 | { |
1152 | - NsmThreadSessionList::AutoLock lock(g_NsmSystem.Sessions); | |
1153 | - const NsmSessionList &p = *lock.List(); | |
1152 | + NsmThreadSessionList::ConstAutoLock lock(g_NsmSystem.Sessions); | |
1153 | + const NsmSessionList &p = lock.List(); | |
1154 | 1154 | |
1155 | 1155 | NsmSessionList::const_iterator it = |
1156 | 1156 | std::find_if(p.begin(), p.end(), NsmSession::FindByHandle(reinterpret_cast<NsmSession *>(wParam))); |
@@ -1198,8 +1198,8 @@ | ||
1198 | 1198 | |
1199 | 1199 | LONG_PTR Service::DoEnumSessionMembers(const WPARAM wParam,const LPARAM lParam) |
1200 | 1200 | { |
1201 | - NsmThreadSessionList::AutoLock lock(g_NsmSystem.Sessions); | |
1202 | - const NsmSessionList &p = *lock.List(); | |
1201 | + NsmThreadSessionList::ConstAutoLock lock(g_NsmSystem.Sessions); | |
1202 | + const NsmSessionList &p = lock.List(); | |
1203 | 1203 | |
1204 | 1204 | NsmSessionList::const_iterator it = |
1205 | 1205 | std::find_if(p.begin(), p.end(), NsmSession::FindByHandle(reinterpret_cast<NsmSession *>(wParam))); |
@@ -1218,8 +1218,8 @@ | ||
1218 | 1218 | LONG_PTR Service::DoSendMessage(const WPARAM wParam,const LPARAM lParam) |
1219 | 1219 | { |
1220 | 1220 | { |
1221 | - NsmThreadSessionList::AutoLock lock(g_NsmSystem.Sessions); | |
1222 | - const NsmSessionList &p = *lock.List(); | |
1221 | + NsmThreadSessionList::ConstAutoLock lock(g_NsmSystem.Sessions); | |
1222 | + const NsmSessionList &p = lock.List(); | |
1223 | 1223 | |
1224 | 1224 | NsmSessionList::const_iterator it = |
1225 | 1225 | std::find_if(p.begin(), p.end(), NsmSession::FindByHandle(reinterpret_cast<NsmSession *>(wParam))); |
@@ -1236,8 +1236,8 @@ | ||
1236 | 1236 | LONG_PTR Service::DoReceiveMessage(const WPARAM wParam,const LPARAM lParam) |
1237 | 1237 | { |
1238 | 1238 | { |
1239 | - NsmThreadSessionList::AutoLock lock(g_NsmSystem.Sessions); | |
1240 | - const NsmSessionList &p = *lock.List(); | |
1239 | + NsmThreadSessionList::ConstAutoLock lock(g_NsmSystem.Sessions); | |
1240 | + const NsmSessionList &p = lock.List(); | |
1241 | 1241 | |
1242 | 1242 | NsmSessionList::const_iterator it = |
1243 | 1243 | std::find_if(p.begin(), p.end(), NsmSession::FindByHandle(reinterpret_cast<NsmSession *>(wParam))); |
@@ -1255,8 +1255,8 @@ | ||
1255 | 1255 | { |
1256 | 1256 | NsmFileSession *result = NULL; |
1257 | 1257 | { |
1258 | - NsmThreadSessionList::AutoLock lock(g_NsmSystem.Sessions); | |
1259 | - const NsmSessionList &p = *lock.List(); | |
1258 | + NsmThreadSessionList::ConstAutoLock lock(g_NsmSystem.Sessions); | |
1259 | + const NsmSessionList &p = lock.List(); | |
1260 | 1260 | |
1261 | 1261 | NsmSessionList::const_iterator it = |
1262 | 1262 | std::find_if(p.begin(), p.end(), NsmSession::FindByHandle(reinterpret_cast<NsmSession *>(wParam))); |
@@ -1264,7 +1264,7 @@ | ||
1264 | 1264 | return 0; |
1265 | 1265 | |
1266 | 1266 | NsmThreadFileSessionList::AutoLock lock2(g_NsmSystem.FileSessions); |
1267 | - NsmFileSessionList &p2 = *lock2.List(); | |
1267 | + NsmFileSessionList &p2 = lock2.List(); | |
1268 | 1268 | |
1269 | 1269 | result = new NsmFileSession(); |
1270 | 1270 |
@@ -1285,7 +1285,7 @@ | ||
1285 | 1285 | { |
1286 | 1286 | { |
1287 | 1287 | NsmThreadFileSessionList::AutoLock lock(g_NsmSystem.FileSessions); |
1288 | - NsmFileSessionList &p = *lock.List(); | |
1288 | + NsmFileSessionList &p = lock.List(); | |
1289 | 1289 | NsmFileSessionList::iterator it |
1290 | 1290 | = std::find_if(p.begin(), p.end(), NsmFileSession::FindByHandle(reinterpret_cast<NsmFileSession *>(wParam))); |
1291 | 1291 | if (it == p.end()) |
@@ -1301,8 +1301,8 @@ | ||
1301 | 1301 | |
1302 | 1302 | LONG_PTR Service::DoGetFileSessionInfo(const WPARAM wParam,const LPARAM lParam) |
1303 | 1303 | { |
1304 | - NsmThreadFileSessionList::AutoLock lock(g_NsmSystem.FileSessions); | |
1305 | - const NsmFileSessionList &p = *lock.List(); | |
1304 | + NsmThreadFileSessionList::ConstAutoLock lock(g_NsmSystem.FileSessions); | |
1305 | + const NsmFileSessionList &p = lock.List(); | |
1306 | 1306 | NsmFileSessionList::const_iterator it = |
1307 | 1307 | std::find_if(p.begin(), p.end(), NsmFileSession::FindByHandle(reinterpret_cast<NsmFileSession *>(wParam))); |
1308 | 1308 | if (it == p.end()) |
@@ -1338,8 +1338,8 @@ | ||
1338 | 1338 | LONG_PTR Service::DoSetFileSessionInfo(const WPARAM wParam,const LPARAM lParam) |
1339 | 1339 | { |
1340 | 1340 | { |
1341 | - NsmThreadFileSessionList::AutoLock lock(g_NsmSystem.FileSessions); | |
1342 | - const NsmFileSessionList &p = *lock.List(); | |
1341 | + NsmThreadFileSessionList::ConstAutoLock lock(g_NsmSystem.FileSessions); | |
1342 | + const NsmFileSessionList &p = lock.List(); | |
1343 | 1343 | |
1344 | 1344 | NsmFileSessionList::const_iterator it = |
1345 | 1345 | std::find_if(p.begin(), p.end(), NsmFileSession::FindByHandle(reinterpret_cast<NsmFileSession *>(wParam))); |
@@ -1382,10 +1382,10 @@ | ||
1382 | 1382 | |
1383 | 1383 | LONG_PTR Service::DoEnumFileSessions(const WPARAM wParam,const LPARAM lParam) |
1384 | 1384 | { |
1385 | - NsmThreadFileSessionList::AutoLock lock(g_NsmSystem.FileSessions); | |
1386 | - const NsmFileSessionList &ls = *lock.List(); | |
1385 | + NsmThreadFileSessionList::ConstAutoLock lock(g_NsmSystem.FileSessions); | |
1386 | + const NsmFileSessionList &p = lock.List(); | |
1387 | 1387 | |
1388 | - for(NsmFileSessionList::const_iterator it = ls.begin();it != ls.end();++it) | |
1388 | + for(NsmFileSessionList::const_iterator it = p.begin();it != p.end();++it) | |
1389 | 1389 | reinterpret_cast<TEnumFileSessionCallback>(wParam)(reinterpret_cast<HNsmFileSession>(it->get()), lParam); |
1390 | 1390 | |
1391 | 1391 | return 0; |
@@ -1395,8 +1395,8 @@ | ||
1395 | 1395 | |
1396 | 1396 | void Service::DoEnumService(const std::tstring &requiredService,const TEnumUIServiceInfo *const pin) |
1397 | 1397 | { |
1398 | - NsmThreadServiceList::AutoLock lock(NsmCom.ServiceList); | |
1399 | - const NsmServiceList &p = *lock.List(); | |
1398 | + NsmThreadServiceList::ConstAutoLock lock(NsmCom.ServiceList); | |
1399 | + const NsmServiceList &p = lock.List(); | |
1400 | 1400 | |
1401 | 1401 | for (NsmServiceList::const_iterator it = p.begin(); it != p.end(); ++it) |
1402 | 1402 | { |
@@ -384,8 +384,8 @@ | ||
384 | 384 | item.mask = LVIF_TEXT; |
385 | 385 | item.iItem = 0; |
386 | 386 | |
387 | - NsmThreadServiceList::AutoLock lock(NsmCom.ServiceList); | |
388 | - const NsmServiceList &p = *lock.List(); | |
387 | + NsmThreadServiceList::ConstAutoLock lock(NsmCom.ServiceList); | |
388 | + const NsmServiceList &p = lock.List(); | |
389 | 389 | |
390 | 390 | for(NsmServiceList::const_iterator it = p.begin();it!=p.end();++item.iItem,++it) |
391 | 391 | { |
@@ -410,8 +410,8 @@ | ||
410 | 410 | item.mask = LVIF_TEXT; |
411 | 411 | item.iItem = 0; |
412 | 412 | |
413 | - NsmThreadEventList::AutoLock lock(NsmCom.EventList); | |
414 | - const NsmEventList &p = *lock.List(); | |
413 | + NsmThreadEventList::ConstAutoLock lock(NsmCom.EventList); | |
414 | + const NsmEventList &p = lock.List(); | |
415 | 415 | |
416 | 416 | for(NsmEventList::const_iterator it = p.begin();it != p.end();++item.iItem,++it) |
417 | 417 | { |
@@ -436,8 +436,8 @@ | ||
436 | 436 | item.mask = LVIF_TEXT; |
437 | 437 | item.iItem = 0; |
438 | 438 | |
439 | - NsmThreadConnectionList::AutoLock lock(g_NsmSystem.Connections); | |
440 | - const NsmConnectionList &p = *lock.List(); | |
439 | + NsmThreadConnectionList::ConstAutoLock lock(g_NsmSystem.Connections); | |
440 | + const NsmConnectionList &p = lock.List(); | |
441 | 441 | |
442 | 442 | for(NsmConnectionList::const_iterator it = p.begin();it != p.end();++item.iItem,++it) |
443 | 443 | { |
@@ -471,8 +471,8 @@ | ||
471 | 471 | item.mask = LVIF_TEXT; |
472 | 472 | item.iItem = 0; |
473 | 473 | |
474 | - NsmThreadSessionList::AutoLock lock(g_NsmSystem.Sessions); | |
475 | - const NsmSessionList &p = *lock.List(); | |
474 | + NsmThreadSessionList::ConstAutoLock lock(g_NsmSystem.Sessions); | |
475 | + const NsmSessionList &p = lock.List(); | |
476 | 476 | |
477 | 477 | for(NsmSessionList::const_iterator it=p.begin();it!=p.end();++item.iItem,++it) |
478 | 478 | { |
@@ -39,14 +39,14 @@ | ||
39 | 39 | ~TNsmCom(); |
40 | 40 | |
41 | 41 | HNsmService CreateNsmService(const std::tstring &serviceName, TNsmServiceProc procAddr); |
42 | - HNsmService GetService(const std::tstring &serviceName); | |
42 | + HNsmService GetService(const std::tstring &serviceName) const; | |
43 | 43 | LONG_PTR CallService(const HNsmService serviceHandle, const WPARAM wParam, const LPARAM lParam) const; |
44 | 44 | |
45 | 45 | HNsmEvent CreateNsmEvent(const std::tstring &eventName); |
46 | - HNsmEvent GetEvent(const std::tstring &eventName); | |
46 | + HNsmEvent GetEvent(const std::tstring &eventName) const; | |
47 | 47 | int NotifyEvent(const HNsmEvent eventHandle, const WPARAM wParam, const LPARAM lParam) const; |
48 | - int HookEvent(const std::tstring &eventName, TNsmEventProc procAddr); | |
49 | - int UnhookEvent(const std::tstring &eventName, TNsmEventProc procAddr); | |
48 | + int HookEvent(const std::tstring &eventName, TNsmEventProc procAddr) const; | |
49 | + int UnhookEvent(const std::tstring &eventName, TNsmEventProc procAddr) const; | |
50 | 50 | }; |
51 | 51 | extern TNsmCom NsmCom; |
52 | 52 |
@@ -96,9 +96,9 @@ | ||
96 | 96 | GetSetInfo->SetConnectionInfo(cHandle, NMCI_USER_MAILSTATUS, 0); |
97 | 97 | |
98 | 98 | MsnThreadConnectionList::AutoLock lock(m_Connections); |
99 | - lock.List()->push_back(MsnConnectionListSub()); | |
99 | + lock.List().push_back(MsnConnectionListSub()); | |
100 | 100 | |
101 | - MsnConnectionListSub Connection = lock.List()->back(); | |
101 | + MsnConnectionListSub Connection = lock.List().back(); | |
102 | 102 | Connection; |
103 | 103 | |
104 | 104 | // SignIn(); |