Unified Diff: src/plugin/PluginDebug.cpp
Issue 11013110:
Cleanup (Closed)
Patch Set: SetPref/GetPref type safety. Comments addressed.
Created July 22, 2013, 12:42 a.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« src/plugin/AdblockPlusClient.cpp ('K')
|
« src/plugin/AdblockPlusClient.cpp ('k')
|
src/plugin/PluginSettings.h » ('j')
|
src/shared/Communication.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: src/plugin/PluginDebug.cpp |
| =================================================================== |
| --- a/src/plugin/PluginDebug.cpp |
| +++ b/src/plugin/PluginDebug.cpp |
| @@ -44,6 +44,12 @@ |
| bool isWorkingThread = settings->IsWorkingThread(dwThreadId); |
| CString processor; |
| + WCHAR tmp[10]; |
|
Wladimir Palant
2013/07/22 06:43:11
I would rather use wchar_t rather than WCHAR - tha
|
| + _itow_s<10>(::GetCurrentProcessId(), tmp, 10); |
|
Wladimir Palant
2013/07/22 06:43:11
I think you can leave out the template parameter h
|
| + if (isWorkingThread) |
| + processor = L"tab" + CString(tmp) + L"_thread"; |
| + else |
| + processor = L"tab" + ::CString(tmp) + L"_ui"; |
|
Wladimir Palant
2013/07/22 06:43:11
CString() or ::CString()? Please choose one ;)
|
| #else |
| if (dwProcessId == 0) |
| { |
« src/plugin/AdblockPlusClient.cpp ('K')
|
« src/plugin/AdblockPlusClient.cpp ('k')
|
src/plugin/PluginSettings.h » ('j')
|
src/shared/Communication.h » ('J')
