summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGeorge Hazan <george.hazan@gmail.com>2012-12-01 19:42:19 +0000
committerGeorge Hazan <george.hazan@gmail.com>2012-12-01 19:42:19 +0000
commit7300094417302d82aa4f3f6e469f91d8facf4f26 (patch)
treee2effcdc0568876d9327206e68838172bac51dfc
parent464d30983285ab816de7e2ac3d72fe5c4a9544e6 (diff)
- mir_icons.h removed from project;
- option "Autp chamge locale" turned on by default; - code cleaning git-svn-id: http://svn.miranda-ng.org/main/trunk@2597 1316c22d-e87f-b044-9b9b-93d7a3e3ba9c
-rw-r--r--plugins/SpellChecker/spellchecker.vcxproj1
-rw-r--r--plugins/SpellChecker/spellchecker.vcxproj.filters3
-rw-r--r--plugins/SpellChecker/src/dictionary.cpp2
-rw-r--r--plugins/SpellChecker/src/options.cpp31
-rw-r--r--plugins/SpellChecker/src/options.h9
-rw-r--r--plugins/SpellChecker/src/utils.cpp516
-rw-r--r--plugins/Utils/mir_options.cpp2
7 files changed, 173 insertions, 391 deletions
diff --git a/plugins/SpellChecker/spellchecker.vcxproj b/plugins/SpellChecker/spellchecker.vcxproj
index 0c4728e1cf..29e5a67298 100644
--- a/plugins/SpellChecker/spellchecker.vcxproj
+++ b/plugins/SpellChecker/spellchecker.vcxproj
@@ -183,7 +183,6 @@
<ClInclude Include="src\autoreplace.h" />
<ClInclude Include="src\commons.h" />
<ClInclude Include="src\dictionary.h" />
- <ClInclude Include="..\utils\mir_icons.h" />
<ClInclude Include="..\utils\mir_memory.h" />
<ClInclude Include="..\utils\mir_options.h" />
<ClInclude Include="src\options.h" />
diff --git a/plugins/SpellChecker/spellchecker.vcxproj.filters b/plugins/SpellChecker/spellchecker.vcxproj.filters
index 0351643dd1..968a037bb1 100644
--- a/plugins/SpellChecker/spellchecker.vcxproj.filters
+++ b/plugins/SpellChecker/spellchecker.vcxproj.filters
@@ -33,9 +33,6 @@
<ClInclude Include="src\dictionary.h">
<Filter>Header Files</Filter>
</ClInclude>
- <ClInclude Include="..\utils\mir_icons.h">
- <Filter>Header Files</Filter>
- </ClInclude>
<ClInclude Include="..\utils\mir_memory.h">
<Filter>Header Files</Filter>
</ClInclude>
diff --git a/plugins/SpellChecker/src/dictionary.cpp b/plugins/SpellChecker/src/dictionary.cpp
index 6f64190e2b..40feda12ef 100644
--- a/plugins/SpellChecker/src/dictionary.cpp
+++ b/plugins/SpellChecker/src/dictionary.cpp
@@ -516,7 +516,7 @@ void GetDictsInfo(LIST<Dictionary> &dicts)
char lang[128];
WideCharToMultiByte(CP_ACP, 0, dict->language, -1, lang, sizeof(lang), NULL, NULL);
- if (!DBGetContactSettingTString(NULL, MODULE_NAME, lang, &dbv))
+ if (!db_get_ts(NULL, MODULE_NAME, lang, &dbv))
{
lstrcpyn(dict->localized_name, dbv.ptszVal, SIZEOF(dict->localized_name));
diff --git a/plugins/SpellChecker/src/options.cpp b/plugins/SpellChecker/src/options.cpp
index 7b462e072e..02033a6988 100644
--- a/plugins/SpellChecker/src/options.cpp
+++ b/plugins/SpellChecker/src/options.cpp
@@ -28,24 +28,22 @@ HANDLE hOptHook = NULL;
Options opts;
-
static INT_PTR CALLBACK OptionsDlgProc(HWND hwndDlg, UINT msg, WPARAM wParam, LPARAM lParam);
static INT_PTR CALLBACK AutoreplaceDlgProc(HWND hwndDlg, UINT msg, WPARAM wParam, LPARAM lParam);
-
static OptPageControl optionsControls[] = {
- { &opts.auto_replace_dict, CONTROL_CHECKBOX, IDC_AUTO_DICT, "AutoReplaceDict", FALSE },
- { &opts.ignore_with_numbers, CONTROL_CHECKBOX, IDC_IGNORE_NUMBERS, "IgnoreWithNumbers", FALSE },
- { &opts.ignore_uppercase, CONTROL_CHECKBOX, IDC_IGNORE_UPPERCASE, "IgnoreUppercase", FALSE },
- { &opts.ask_when_sending_with_error, CONTROL_CHECKBOX, IDC_ASK_ON_ERROR, "AskWhenSendingWithError", FALSE },
- { &opts.underline_type, CONTROL_COMBO, IDC_UNDERLINE_TYPE, "UnderlineType", CFU_UNDERLINEWAVE - CFU_UNDERLINEDOUBLE },
- { &opts.cascade_corrections, CONTROL_CHECKBOX, IDC_CASCADE_CORRECTIONS, "CascadeCorrections", FALSE },
- { &opts.show_all_corrections, CONTROL_CHECKBOX, IDC_SHOW_ALL_CORRECTIONS, "ShowAllCorrections", FALSE },
- { &opts.show_wrong_word, CONTROL_CHECKBOX, IDC_SHOW_WRONG_WORD, "ShowWrongWord", TRUE },
- { &opts.use_flags, CONTROL_CHECKBOX, IDC_USE_FLAGS, "UseFlags", TRUE },
- { &opts.auto_locale, CONTROL_CHECKBOX, IDC_AUTO_LOCALE, "AutoLocale", FALSE },
- { &opts.use_other_apps_dicts, CONTROL_CHECKBOX, IDC_OTHER_PROGS, "UseOtherAppsDicts", TRUE },
- { &opts.handle_underscore, CONTROL_CHECKBOX, IDC_HANDLE_UNDERSCORE, "HandleUnderscore", FALSE },
+ { &opts.auto_replace_dict, CONTROL_CHECKBOX, IDC_AUTO_DICT, "AutoReplaceDict", FALSE },
+ { &opts.ignore_with_numbers, CONTROL_CHECKBOX, IDC_IGNORE_NUMBERS, "IgnoreWithNumbers", FALSE },
+ { &opts.ignore_uppercase, CONTROL_CHECKBOX, IDC_IGNORE_UPPERCASE, "IgnoreUppercase", FALSE },
+ { &opts.ask_when_sending_with_error, CONTROL_CHECKBOX, IDC_ASK_ON_ERROR, "AskWhenSendingWithError", FALSE },
+ { &opts.underline_type, CONTROL_COMBO, IDC_UNDERLINE_TYPE, "UnderlineType", CFU_UNDERLINEWAVE - CFU_UNDERLINEDOUBLE },
+ { &opts.cascade_corrections, CONTROL_CHECKBOX, IDC_CASCADE_CORRECTIONS, "CascadeCorrections", FALSE },
+ { &opts.show_all_corrections, CONTROL_CHECKBOX, IDC_SHOW_ALL_CORRECTIONS, "ShowAllCorrections", FALSE },
+ { &opts.show_wrong_word, CONTROL_CHECKBOX, IDC_SHOW_WRONG_WORD, "ShowWrongWord", TRUE },
+ { &opts.use_flags, CONTROL_CHECKBOX, IDC_USE_FLAGS, "UseFlags", TRUE },
+ { &opts.auto_locale, CONTROL_CHECKBOX, IDC_AUTO_LOCALE, "AutoLocale", TRUE },
+ { &opts.use_other_apps_dicts, CONTROL_CHECKBOX, IDC_OTHER_PROGS, "UseOtherAppsDicts", TRUE },
+ { &opts.handle_underscore, CONTROL_CHECKBOX, IDC_HANDLE_UNDERSCORE, "HandleUnderscore", FALSE },
};
static UINT optionsExpertControls[] = {
@@ -53,15 +51,12 @@ static UINT optionsExpertControls[] = {
IDC_SHOW_WRONG_WORD, IDC_USE_FLAGS, IDC_AUTO_LOCALE, IDC_OTHER_PROGS, IDC_HANDLE_UNDERSCORE
};
-
static OptPageControl autoReplaceControls[] = {
{ &opts.auto_replace_user, CONTROL_CHECKBOX, IDC_AUTO_USER, "AutoReplaceUser", TRUE },
};
-
// Functions //////////////////////////////////////////////////////////////////////////////////////
-
int InitOptionsCallback(WPARAM wParam,LPARAM lParam)
{
OPTIONSDIALOGPAGE odp = { 0 };
@@ -113,7 +108,7 @@ void LoadOptions()
}
DBVARIANT dbv;
- if (!DBGetContactSettingTString(NULL, MODULE_NAME, "DefaultLanguage", &dbv))
+ if (!db_get_ts(NULL, MODULE_NAME, "DefaultLanguage", &dbv))
{
lstrcpyn(opts.default_language, dbv.ptszVal, SIZEOF(opts.default_language));
DBFreeVariant(&dbv);
diff --git a/plugins/SpellChecker/src/options.h b/plugins/SpellChecker/src/options.h
index 70710fdb9d..073864cdd7 100644
--- a/plugins/SpellChecker/src/options.h
+++ b/plugins/SpellChecker/src/options.h
@@ -21,7 +21,6 @@ Boston, MA 02111-1307, USA.
#ifndef __OPTIONS_H__
# define __OPTIONS_H__
-
#define POPUP_ACTION_DONOTHING 0
#define POPUP_ACTION_CLOSEPOPUP 1
#define POPUP_ACTION_OPENHISTORY 2
@@ -30,8 +29,8 @@ Boston, MA 02111-1307, USA.
#define POPUP_DELAY_CUSTOM 1
#define POPUP_DELAY_PERMANENT 2
-
-struct Options {
+struct Options
+{
TCHAR default_language[32];
BOOL auto_replace_dict;
BOOL auto_replace_user;
@@ -51,18 +50,14 @@ struct Options {
extern Options opts;
-
// Initializations needed by options
void InitOptions();
// Deinitializations needed by options
void DeInitOptions();
-
// Loads the options from DB
// It don't need to be called, except in some rare cases
void LoadOptions();
-
-
#endif // __OPTIONS_H__
diff --git a/plugins/SpellChecker/src/utils.cpp b/plugins/SpellChecker/src/utils.cpp
index d8e5f5706e..bcb3fde5a8 100644
--- a/plugins/SpellChecker/src/utils.cpp
+++ b/plugins/SpellChecker/src/utils.cpp
@@ -39,7 +39,6 @@ void SetUnderline(Dialog *dlg, int pos_start, int pos_end)
dlg->markedSomeWord = TRUE;
}
-
BOOL IsMyUnderline(const CHARFORMAT2 &cf)
{
return (cf.dwEffects & CFE_UNDERLINE)
@@ -48,13 +47,10 @@ BOOL IsMyUnderline(const CHARFORMAT2 &cf)
&& (cf.bUnderlineType & ~0x0F) == 0x50;
}
-
void SetNoUnderline(RichEdit *re, int pos_start, int pos_end)
{
- if (opts.handle_underscore)
- {
- for(int i = pos_start; i <= pos_end; i++)
- {
+ if (opts.handle_underscore) {
+ for(int i = pos_start; i <= pos_end; i++) {
re->SetSel(i, min(i+1, pos_end));
CHARFORMAT2 cf;
@@ -62,8 +58,7 @@ void SetNoUnderline(RichEdit *re, int pos_start, int pos_end)
re->SendMessage(EM_GETCHARFORMAT, (WPARAM) SCF_SELECTION, (LPARAM)&cf);
BOOL mine = IsMyUnderline(cf);
- if (mine)
- {
+ if (mine) {
cf.cbSize = sizeof(CHARFORMAT2);
cf.dwMask = CFM_UNDERLINE | CFM_UNDERLINETYPE;
cf.dwEffects = 0;
@@ -72,8 +67,7 @@ void SetNoUnderline(RichEdit *re, int pos_start, int pos_end)
}
}
}
- else
- {
+ else {
re->SetSel(pos_start, pos_end);
CHARFORMAT2 cf;
@@ -85,7 +79,6 @@ void SetNoUnderline(RichEdit *re, int pos_start, int pos_end)
}
}
-
void SetNoUnderline(Dialog *dlg)
{
dlg->re->Stop();
@@ -94,13 +87,11 @@ void SetNoUnderline(Dialog *dlg)
dlg->re->Start();
}
-
inline BOOL IsNumber(TCHAR c)
{
return c >= _T('0') && c <= _T('9');
}
-
inline BOOL IsURL(TCHAR c)
{
return (c >= _T('a') && c <= _T('z'))
@@ -114,7 +105,6 @@ inline BOOL IsURL(TCHAR c)
|| c == _T('@') || c == _T('#');
}
-
int FindURLEnd(Dialog *dlg, TCHAR *text, int start_pos, int *checked_until = NULL)
{
int num_slashes = 0;
@@ -123,10 +113,8 @@ int FindURLEnd(Dialog *dlg, TCHAR *text, int start_pos, int *checked_until = NUL
int i = start_pos;
- for (; IsURL(text[i]) || dlg->lang->isWordChar(text[i]); i++)
- {
+ for (; IsURL(text[i]) || dlg->lang->isWordChar(text[i]); i++) {
TCHAR c = text[i];
-
if (c == _T('\\') || c == _T('/'))
num_slashes++;
else if (c == _T('.'))
@@ -164,7 +152,6 @@ int ReplaceWord(Dialog *dlg, CHARRANGE &sel, TCHAR *new_word)
return dif;
}
-
class TextParser
{
public:
@@ -177,7 +164,6 @@ public:
virtual void deal(const TCHAR *text, bool *mark, bool *replace, TCHAR **replacement) =0;
};
-
class SpellParser : public TextParser
{
Dictionary *dict;
@@ -199,8 +185,7 @@ public:
bool feed(int pos, TCHAR c)
{
// Is inside a word?
- if (dict->isWordChar(c) || IsNumber(c))
- {
+ if (dict->isWordChar(c) || IsNumber(c)) {
if (last_pos == -1)
last_pos = pos;
@@ -231,11 +216,9 @@ public:
return;
// Has to auto-correct?
- if (opts.auto_replace_dict)
- {
+ if (opts.auto_replace_dict) {
*replacement = dict->autoSuggestOne(text);
- if (*replacement != NULL)
- {
+ if (*replacement != NULL) {
*replace = true;
return;
}
@@ -245,7 +228,6 @@ public:
}
};
-
class AutoReplaceParser : public TextParser
{
AutoReplaceMap *ar;
@@ -265,8 +247,7 @@ public:
bool feed(int pos, TCHAR c)
{
// Is inside a word?
- if (ar->isWordChar(c))
- {
+ if (ar->isWordChar(c)) {
if (last_pos == -1)
last_pos = pos;
return false;
@@ -300,21 +281,17 @@ int CheckTextLine(Dialog *dlg, int line, TextParser *parser,
// Now lets get the words
int next_char_for_url = 0;
- for (int pos = 0; pos < len; pos++)
- {
+ for (int pos = 0; pos < len; pos++) {
int url_end = pos;
- if (pos >= next_char_for_url)
- {
+ if (pos >= next_char_for_url) {
url_end = FindURLEnd(dlg, text, pos, &next_char_for_url);
next_char_for_url++;
}
- if (url_end > pos)
- {
+ if (url_end > pos) {
BOOL ignore_url = FALSE;
- if (test_urls)
- {
+ if (test_urls) {
// All the url must be handled by the parser
parser->reset();
@@ -325,25 +302,20 @@ int CheckTextLine(Dialog *dlg, int line, TextParser *parser,
if (feed || parser->getFirstCharPos() != pos)
ignore_url = TRUE;
}
- else
- ignore_url = TRUE;
+ else ignore_url = TRUE;
pos = url_end;
- if (ignore_url)
- {
+ if (ignore_url) {
parser->reset();
continue;
}
}
- else
- {
+ else {
TCHAR c = text[pos];
BOOL feed = parser->feed(pos, c);
-
- if (!feed)
- {
+ if (!feed) {
if (pos >= len-1)
pos = len; // To check the last block
else
@@ -364,8 +336,7 @@ int CheckTextLine(Dialog *dlg, int line, TextParser *parser,
if (sel.cpMin <= ignored.cpMax && sel.cpMax >= ignored.cpMin)
continue;
- if (ignore_upper)
- {
+ if (ignore_upper) {
BOOL upper = TRUE;
for(int i = last_pos; i < pos && upper; i++)
upper = !IsCharLower(text[i]);
@@ -373,8 +344,7 @@ int CheckTextLine(Dialog *dlg, int line, TextParser *parser,
continue;
}
- if (ignore_with_numbers)
- {
+ if (ignore_with_numbers) {
BOOL hasNumbers = FALSE;
for(int i = last_pos; i < pos && !hasNumbers; i++)
hasNumbers = IsNumber(text[i]);
@@ -389,12 +359,10 @@ int CheckTextLine(Dialog *dlg, int line, TextParser *parser,
TCHAR *replacement = NULL;
parser->deal(&text[last_pos], &mark, &replace, &replacement);
- if (replace)
- {
+ if (replace) {
// Replace in rich edit
int dif = dlg->re->Replace(sel.cpMin, sel.cpMax, replacement);
- if (dif != 0)
- {
+ if (dif != 0) {
// Read line again
dlg->re->GetLine(line, text, SIZEOF(text));
len = lstrlen(text);
@@ -407,8 +375,7 @@ int CheckTextLine(Dialog *dlg, int line, TextParser *parser,
free(replacement);
}
- else if (mark)
- {
+ else if (mark) {
SetUnderline(dlg, sel.cpMin, sel.cpMax);
if (callback != NULL)
@@ -421,23 +388,19 @@ int CheckTextLine(Dialog *dlg, int line, TextParser *parser,
return errors;
}
-
// Checks for errors in all text
-int CheckText(Dialog *dlg, BOOL check_all,
- FoundWrongWordCallback callback = NULL, void *param = NULL)
+int CheckText(Dialog *dlg, BOOL check_all, FoundWrongWordCallback callback = NULL, void *param = NULL)
{
int errors = 0;
dlg->re->Stop();
- if (dlg->re->GetTextLength() > 0)
- {
+ if (dlg->re->GetTextLength() > 0) {
int lines = dlg->re->GetLineCount();
int line = 0;
CHARRANGE cur_sel = { -1, -1 };
- if (!check_all)
- {
+ if (!check_all) {
// Check only the current line, one up and one down
int current_line = dlg->re->GetLineFromChar(dlg->re->GetSel().cpMin);
line = max(line, current_line - 1);
@@ -445,18 +408,15 @@ int CheckText(Dialog *dlg, BOOL check_all,
cur_sel = dlg->re->GetSel();
}
- for (; line < lines; line++)
- {
+ for (; line < lines; line++) {
int first_char = dlg->re->GetFirstCharOfLine(line);
SetNoUnderline(dlg->re, first_char, first_char + dlg->re->GetLineLength(line));
- if (opts.auto_replace_user)
- {
+ if (opts.auto_replace_user)
errors += CheckTextLine(dlg, line, &AutoReplaceParser(dlg->lang->autoReplace),
FALSE, FALSE, TRUE,
cur_sel, callback, param);
- }
errors += CheckTextLine(dlg, line, &SpellParser(dlg->lang),
opts.ignore_uppercase, opts.ignore_with_numbers, FALSE,
@@ -469,25 +429,21 @@ int CheckText(Dialog *dlg, BOOL check_all,
SetNoUnderline(dlg->re, len, len);
dlg->re->Start();
-
return errors;
}
-
void ToLocaleID(TCHAR *szKLName, size_t size)
{
TCHAR *stopped = NULL;
USHORT langID = (USHORT) _tcstol(szKLName, &stopped, 16);
- TCHAR ini[32];
- TCHAR end[32];
+ TCHAR ini[32], end[32];
GetLocaleInfo(MAKELCID(langID, 0), LOCALE_SISO639LANGNAME, ini, SIZEOF(ini));
GetLocaleInfo(MAKELCID(langID, 0), LOCALE_SISO3166CTRYNAME, end, SIZEOF(end));
mir_sntprintf(szKLName, size, _T("%s_%s"), ini, end);
}
-
void LoadDictFromKbdl(Dialog *dlg)
{
TCHAR szKLName[KL_NAMELENGTH + 1];
@@ -497,13 +453,8 @@ void LoadDictFromKbdl(Dialog *dlg)
mir_sntprintf(szKLName, SIZEOF(szKLName), _T("%x"), (int) LOWORD(hkl));
ToLocaleID(szKLName, SIZEOF(szKLName));
- // Old code (use keyboard layout)
-// GetKeyboardLayoutName(szKLName);
-// ToLocaleID(szKLName, SIZEOF(szKLName));
-
int d = GetClosestLanguage(szKLName);
- if (d >= 0)
- {
+ if (d >= 0) {
dlg->lang = languages[d];
dlg->lang->load();
@@ -519,8 +470,7 @@ int TimerCheck(Dialog *dlg, BOOL forceCheck = FALSE)
if (!dlg->enabled || dlg->lang == NULL)
return -1;
- if (!dlg->lang->isLoaded())
- {
+ if (!dlg->lang->isLoaded()) {
SetTimer(dlg->hwnd, TIMER_ID, 500, NULL);
return -1;
}
@@ -539,7 +489,6 @@ int TimerCheck(Dialog *dlg, BOOL forceCheck = FALSE)
return CheckText(dlg, TRUE);
}
-
LRESULT CALLBACK OwnerProc(HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam)
{
DialogMapType::iterator dlgit = dialogs.find(hwnd);
@@ -548,23 +497,17 @@ LRESULT CALLBACK OwnerProc(HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam)
Dialog *dlg = dlgit->second;
- if (msg == WM_COMMAND && (LOWORD(wParam) == IDOK || LOWORD(wParam) == 1624))
- {
- if (opts.ask_when_sending_with_error)
- {
+ if (msg == WM_COMMAND && (LOWORD(wParam) == IDOK || LOWORD(wParam) == 1624)) {
+ if (opts.ask_when_sending_with_error) {
int errors = TimerCheck(dlg, TRUE);
- if (errors > 0)
- {
+ if (errors > 0) {
TCHAR text[500];
mir_sntprintf(text,SIZEOF(text),TranslateT("There are %d spelling errors. Are you sure you want to send this message?"),errors);
if (MessageBox(hwnd,text,TranslateT("Spell Checker"), MB_ICONQUESTION | MB_YESNO) == IDNO)
- {
return TRUE;
- }
}
}
- else if (opts.auto_replace_dict || opts.auto_replace_user)
- {
+ else if (opts.auto_replace_dict || opts.auto_replace_user) {
// Fix all
TimerCheck(dlg);
}
@@ -583,18 +526,14 @@ LRESULT CALLBACK OwnerProc(HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam)
return CallWindowProc(dlg->owner_old_edit_proc, hwnd, msg, wParam, lParam);
}
-
void ToggleEnabled(Dialog *dlg)
{
dlg->enabled = !dlg->enabled;
DBWriteContactSettingByte(dlg->hContact, MODULE_NAME, dlg->name, dlg->enabled);
if (!dlg->enabled)
- {
SetNoUnderline(dlg);
- }
- else
- {
+ else {
dlg->changed = TRUE;
SetTimer(dlg->hwnd, TIMER_ID, 100, NULL);
}
@@ -603,7 +542,6 @@ void ToggleEnabled(Dialog *dlg)
ModifyIcon(dlg);
}
-
LRESULT CALLBACK EditProc(HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam)
{
DialogMapType::iterator dlgit = dialogs.find(hwnd);
@@ -615,41 +553,36 @@ LRESULT CALLBACK EditProc(HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam)
return -1;
// Hotkey support
- {
- MSG msgData = {0};
- msgData.hwnd = hwnd;
- msgData.message = msg;
- msgData.wParam = wParam;
- msgData.lParam = lParam;
-
- int action = CallService(MS_HOTKEY_CHECK, (WPARAM) &msgData, (LPARAM) "Spell Checker");
- if (action == HOTKEY_ACTION_TOGGLE)
- {
- ToggleEnabled(dlg);
- return 1;
- }
+ MSG msgData = {0};
+ msgData.hwnd = hwnd;
+ msgData.message = msg;
+ msgData.wParam = wParam;
+ msgData.lParam = lParam;
+
+ int action = CallService(MS_HOTKEY_CHECK, (WPARAM) &msgData, (LPARAM) "Spell Checker");
+ if (action == HOTKEY_ACTION_TOGGLE) {
+ ToggleEnabled(dlg);
+ return 1;
}
LRESULT ret = CallWindowProc(dlg->old_edit_proc, hwnd, msg, wParam, lParam);
- switch(msg)
- {
- case WM_KEYDOWN:
- {
- if (wParam != VK_DELETE)
- break;
- }
- case WM_CHAR:
- {
- if (dlg->re->IsStopped())
- break;
+ switch(msg) {
+ case WM_KEYDOWN:
+ if (wParam != VK_DELETE)
+ break;
- if (lParam & (1 << 28)) // ALT key
- break;
+ case WM_CHAR:
+ if (dlg->re->IsStopped())
+ break;
- if (GetKeyState(VK_CONTROL) & 0x8000) // CTRL key
- break;
+ if (lParam & (1 << 28)) // ALT key
+ break;
+ if ( GetKeyState(VK_CONTROL) & 0x8000) // CTRL key
+ break;
+
+ {
TCHAR c = (TCHAR) wParam;
BOOL deleting = (c == VK_BACK || c == VK_DELETE);
@@ -669,21 +602,15 @@ LRESULT CALLBACK EditProc(HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam)
if (dlg->re->IsReadOnly())
break;
-
if (!deleting && !dlg->lang->isWordChar(c))
- {
CheckText(dlg, FALSE);
- }
- else
- {
+ else {
// Remove underline of current word
-
CHARFORMAT2 cf;
cf.cbSize = sizeof(CHARFORMAT2);
dlg->re->SendMessage(EM_GETCHARFORMAT, (WPARAM) SCF_SELECTION, (LPARAM)&cf);
- if (IsMyUnderline(cf))
- {
+ if ( IsMyUnderline(cf)) {
dlg->re->Stop();
CHARRANGE sel = dlg->re->GetSel();
@@ -697,63 +624,50 @@ LRESULT CALLBACK EditProc(HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam)
dlg->re->Start();
}
}
-
- break;
}
- case EM_REPLACESEL:
- case WM_SETTEXT:
- case EM_SETTEXTEX:
- case EM_PASTESPECIAL:
- case WM_PASTE:
- {
- if (dlg->re->IsStopped())
- break;
-
- KillTimer(hwnd, TIMER_ID);
- SetTimer(hwnd, TIMER_ID, 100, NULL);
+ break;
- dlg->changed = TRUE;
+ case EM_REPLACESEL:
+ case WM_SETTEXT:
+ case EM_SETTEXTEX:
+ case EM_PASTESPECIAL:
+ case WM_PASTE:
+ if (dlg->re->IsStopped())
break;
- }
- case WM_TIMER:
- {
- if (wParam != TIMER_ID)
- break;
+ KillTimer(hwnd, TIMER_ID);
+ SetTimer(hwnd, TIMER_ID, 100, NULL);
+
+ dlg->changed = TRUE;
+ break;
+ case WM_TIMER:
+ if (wParam == TIMER_ID)
TimerCheck(dlg);
- break;
- }
+ break;
- case WMU_DICT_CHANGED:
- {
+ case WMU_DICT_CHANGED:
+ KillTimer(hwnd, TIMER_ID);
+ SetTimer(hwnd, TIMER_ID, 100, NULL);
+
+ dlg->changed = TRUE;
+ break;
+
+ case WMU_KBDL_CHANGED:
+ if (opts.auto_locale) {
KillTimer(hwnd, TIMER_ID);
SetTimer(hwnd, TIMER_ID, 100, NULL);
dlg->changed = TRUE;
- break;
- }
-
- case WMU_KBDL_CHANGED:
- {
- if (opts.auto_locale)
- {
- KillTimer(hwnd, TIMER_ID);
- SetTimer(hwnd, TIMER_ID, 100, NULL);
-
- dlg->changed = TRUE;
- LoadDictFromKbdl(dlg);
- }
- break;
+ LoadDictFromKbdl(dlg);
}
+ break;
- case WM_INPUTLANGCHANGE:
- {
- // Allow others to process this message and we get only the result
- PostMessage(hwnd, WMU_KBDL_CHANGED, 0, 0);
- break;
- }
+ case WM_INPUTLANGCHANGE:
+ // Allow others to process this message and we get only the result
+ PostMessage(hwnd, WMU_KBDL_CHANGED, 0, 0);
+ break;
}
return ret;
@@ -765,12 +679,8 @@ int GetClosestLanguage(TCHAR *lang_name)
// Search the language by name
for (i = 0; i < languages.getCount(); i++)
- {
if (lstrcmpi(languages[i]->language, lang_name) == 0)
- {
return i;
- }
- }
// Try searching by the prefix only
TCHAR lang[128];
@@ -782,17 +692,12 @@ int GetClosestLanguage(TCHAR *lang_name)
// First check if there is a language that is only the prefix
for (i = 0; i < languages.getCount(); i++)
- {
if (lstrcmpi(languages[i]->language, lang) == 0)
- {
return i;
- }
- }
// Now try any suffix
size_t len = lstrlen(lang);
- for (i = 0; i < languages.getCount(); i++)
- {
+ for (i = 0; i < languages.getCount(); i++) {
TCHAR *p = _tcschr(languages[i]->language, _T('_'));
if (p == NULL)
continue;
@@ -802,9 +707,7 @@ int GetClosestLanguage(TCHAR *lang_name)
continue;
if (_tcsnicmp(languages[i]->language, lang_name, len) == 0)
- {
return i;
- }
}
return -1;
@@ -824,24 +727,17 @@ void GetUserProtoLanguageSetting(Dialog *dlg, HANDLE hContact, char *group, char
int caps = (isProtocol ? CallProtoService(group, PS_GETCAPS, PFLAGNUM_4, 0) : 0);
if (caps & PF4_INFOSETTINGSVC)
- {
rc = CallProtoService(group, PS_GETINFOSETTING, (WPARAM) hContact, (LPARAM) &cgs);
- }
- else
- {
+ else {
rc = CallService(MS_DB_CONTACT_GETSETTING_STR_EX, (WPARAM)hContact, (LPARAM)&cgs);
if (rc == CALLSERVICE_NOTFOUND)
- {
rc = CallService(MS_DB_CONTACT_GETSETTING_STR, (WPARAM)hContact, (LPARAM)&cgs);
- }
}
- if (!rc && dbv.type == DBVT_TCHAR && dbv.ptszVal != NULL)
- {
+ if (!rc && dbv.type == DBVT_TCHAR && dbv.ptszVal != NULL) {
TCHAR *lang = dbv.ptszVal;
- for (int i = 0; i < languages.getCount(); i++)
- {
+ for (int i = 0; i < languages.getCount(); i++) {
Dictionary *dict = languages[i];
if (lstrcmpi(dict->localized_name, lang) == 0
|| lstrcmpi(dict->english_name, lang) == 0
@@ -873,17 +769,13 @@ void GetUserLanguageSetting(Dialog *dlg, char *setting)
// If not found and is inside meta, try to get from the meta
INT_PTR mc = CallService(MS_MC_GETPROTOCOLNAME, 0, 0);
- if (mc != CALLSERVICE_NOTFOUND)
- {
+ if (mc != CALLSERVICE_NOTFOUND) {
char* metacontacts_proto = (char *) mc;
- if (metacontacts_proto != NULL)
- {
+ if (metacontacts_proto != NULL) {
mc = CallService(MS_MC_GETMETACONTACT, (WPARAM) dlg->hContact, 0);
- if (mc != CALLSERVICE_NOTFOUND)
- {
+ if (mc != CALLSERVICE_NOTFOUND) {
HANDLE hMetaContact = (HANDLE) mc;
- if (hMetaContact != NULL)
- {
+ if (hMetaContact != NULL) {
GetUserProtoLanguageSetting(dlg, hMetaContact, metacontacts_proto, setting);
if (dlg->lang_name[0] != _T('\0'))
return;
@@ -901,45 +793,35 @@ void GetContactLanguage(Dialog *dlg)
dlg->lang_name[0] = _T('\0');
- if (dlg->hContact == NULL)
- {
- if (!DBGetContactSettingTString(NULL, MODULE_NAME, dlg->name, &dbv))
- {
+ if (dlg->hContact == NULL) {
+ if ( !db_get_ts(NULL, MODULE_NAME, dlg->name, &dbv)) {
lstrcpyn(dlg->lang_name, dbv.ptszVal, SIZEOF(dlg->lang_name));
DBFreeVariant(&dbv);
}
}
- else
- {
- if (!DBGetContactSettingTString(dlg->hContact, MODULE_NAME, "TalkLanguage", &dbv))
- {
+ else {
+ if (!db_get_ts(dlg->hContact, MODULE_NAME, "TalkLanguage", &dbv)) {
lstrcpyn(dlg->lang_name, dbv.ptszVal, SIZEOF(dlg->lang_name));
DBFreeVariant(&dbv);
}
- if (dlg->lang_name[0] == _T('\0') && !DBGetContactSettingTString(dlg->hContact, "eSpeak", "TalkLanguage", &dbv))
- {
+ if (dlg->lang_name[0] == _T('\0') && !db_get_ts(dlg->hContact, "eSpeak", "TalkLanguage", &dbv)) {
lstrcpyn(dlg->lang_name, dbv.ptszVal, SIZEOF(dlg->lang_name));
DBFreeVariant(&dbv);
}
// Try from metacontact
- if (dlg->lang_name[0] == _T('\0'))
- {
+ if (dlg->lang_name[0] == _T('\0')) {
INT_PTR mc = CallService(MS_MC_GETMETACONTACT, (WPARAM) dlg->hContact, 0);
- if (mc != CALLSERVICE_NOTFOUND)
- {
+ if (mc != CALLSERVICE_NOTFOUND) {
HANDLE hMetaContact = (HANDLE) mc;
- if (hMetaContact != NULL)
- {
- if (!DBGetContactSettingTString(hMetaContact, MODULE_NAME, "TalkLanguage", &dbv))
- {
+ if (hMetaContact != NULL) {
+ if (!db_get_ts(hMetaContact, MODULE_NAME, "TalkLanguage", &dbv)) {
lstrcpyn(dlg->lang_name, dbv.ptszVal, SIZEOF(dlg->lang_name));
DBFreeVariant(&dbv);
}
- if (dlg->lang_name[0] == _T('\0') && !DBGetContactSettingTString(hMetaContact, "eSpeak", "TalkLanguage", &dbv))
- {
+ if (dlg->lang_name[0] == _T('\0') && !db_get_ts(hMetaContact, "eSpeak", "TalkLanguage", &dbv)) {
lstrcpyn(dlg->lang_name, dbv.ptszVal, SIZEOF(dlg->lang_name));
DBFreeVariant(&dbv);
}
@@ -963,34 +845,27 @@ void GetContactLanguage(Dialog *dlg)
}
int i = GetClosestLanguage(dlg->lang_name);
- if (i < 0)
- {
+ if (i < 0) {
// Lost a dict?
lstrcpyn(dlg->lang_name, opts.default_language, SIZEOF(dlg->lang_name));
i = GetClosestLanguage(dlg->lang_name);
}
- if (i >= 0)
- {
+ if (i >= 0) {
dlg->lang = languages[i];
dlg->lang->load();
}
- else
- {
- dlg->lang = NULL;
- }
+ else dlg->lang = NULL;
}
void ModifyIcon(Dialog *dlg)
{
- if (ServiceExists(MS_MSG_MODIFYICON))
- {
+ if ( ServiceExists(MS_MSG_MODIFYICON)) {
StatusIconData sid = {0};
sid.cbSize = sizeof(sid);
sid.szModule = MODULE_NAME;
- for (int i = 0; i < languages.getCount(); i++)
- {
+ for (int i = 0; i < languages.getCount(); i++) {
sid.dwId = i;
if (languages[i] == dlg->lang)
@@ -1012,11 +887,9 @@ INT_PTR AddContactTextBoxService(WPARAM wParam, LPARAM lParam)
return AddContactTextBox(sci->hContact, sci->hwnd, sci->window_name, FALSE, NULL);
}
-
void NotifyWrongSRMM()
{
static BOOL notified = FALSE;
-
if (notified)
return;
@@ -1027,21 +900,18 @@ void NotifyWrongSRMM()
notified = TRUE;
}
-
int AddContactTextBox(HANDLE hContact, HWND hwnd, char *name, BOOL srmm, HWND hwndOwner)
{
if (languages.getCount() <= 0)
return 0;
- if (dialogs.find(hwnd) == dialogs.end())
- {
+ if (dialogs.find(hwnd) == dialogs.end()) {
// Fill dialog data
Dialog *dlg = (Dialog *) malloc(sizeof(Dialog));
ZeroMemory(dlg, sizeof(Dialog));
dlg->re = new RichEdit(hwnd);
- if (!dlg->re->IsValid())
- {
+ if (!dlg->re->IsValid()) {
delete dlg->re;
free(dlg);
@@ -1065,8 +935,7 @@ int AddContactTextBox(HANDLE hContact, HWND hwnd, char *name, BOOL srmm, HWND hw
dlg->old_edit_proc = (WNDPROC) SetWindowLongPtr(dlg->hwnd, GWLP_WNDPROC, (LONG_PTR) EditProc);
dialogs[hwnd] = dlg;
- if (dlg->srmm && hwndOwner != NULL)
- {
+ if (dlg->srmm && hwndOwner != NULL) {
dlg->hwnd_owner = hwndOwner;
dlg->owner_old_edit_proc = (WNDPROC) SetWindowLongPtr(dlg->hwnd_owner, GWLP_WNDPROC, (LONG_PTR) OwnerProc);
dialogs[dlg->hwnd_owner] = dlg;
@@ -1098,10 +967,8 @@ void FreePopupData(Dialog *dlg)
menus.erase(dlg->hwnd_menu_owner);
dlg->hwnd_menu_owner = NULL;
- if (dlg->wrong_words != NULL)
- {
- for (unsigned i = 0; i < dlg->wrong_words->size(); i++)
- {
+ if (dlg->wrong_words != NULL) {
+ for (unsigned i = 0; i < dlg->wrong_words->size(); i++) {
FREE((*dlg->wrong_words)[i].word)
DESTROY_MENY((*dlg->wrong_words)[i].hMeSubMenu)
@@ -1116,7 +983,6 @@ void FreePopupData(Dialog *dlg)
}
}
-
INT_PTR RemoveContactTextBoxService(WPARAM wParam, LPARAM lParam)
{
HWND hwnd = (HWND) wParam;
@@ -1126,12 +992,10 @@ INT_PTR RemoveContactTextBoxService(WPARAM wParam, LPARAM lParam)
return RemoveContactTextBox(hwnd);
}
-
int RemoveContactTextBox(HWND hwnd)
{
DialogMapType::iterator dlgit = dialogs.find(hwnd);
- if (dlgit != dialogs.end())
- {
+ if (dlgit != dialogs.end()) {
Dialog *dlg = dlgit->second;
KillTimer(hwnd, TIMER_ID);
@@ -1140,8 +1004,7 @@ int RemoveContactTextBox(HWND hwnd)
SetWindowLongPtr(hwnd, GWLP_WNDPROC, (LONG_PTR) dlg->old_edit_proc);
dialogs.erase(hwnd);
- if (dlg->hwnd_owner != NULL)
- {
+ if (dlg->hwnd_owner != NULL) {
if (dlg->owner_old_edit_proc != NULL)
SetWindowLongPtr(dlg->hwnd_owner, GWLP_WNDPROC, (LONG_PTR) dlg->owner_old_edit_proc);
dialogs.erase(dlg->hwnd_owner);
@@ -1155,7 +1018,6 @@ int RemoveContactTextBox(HWND hwnd)
return 0;
}
-
// TODO Make this better
BOOL GetWordCharRange(Dialog *dlg, CHARRANGE &sel, TCHAR *text, size_t text_len, int &first_char)
{
@@ -1220,8 +1082,7 @@ void AppendSubmenu(HMENU hMenu, HMENU hSubMenu, TCHAR *name)
mii.hSubMenu = hSubMenu;
mii.dwTypeData = name;
mii.cch = lstrlen(name);
- int ret = InsertMenuItem(hMenu, 0, TRUE, &mii);
-
+ InsertMenuItem(hMenu, 0, TRUE, &mii);
}
void AppendMenuItem(HMENU hMenu, int id, TCHAR *name, HICON hIcon, BOOL checked)
@@ -1239,12 +1100,9 @@ void AppendMenuItem(HMENU hMenu, int id, TCHAR *name, HICON hIcon, BOOL checked)
mii.hbmpUnchecked = iconInfo.hbmColor;
mii.dwTypeData = name;
mii.cch = lstrlen(name);
- int ret = InsertMenuItem(hMenu, 0, TRUE, &mii);
+ InsertMenuItem(hMenu, 0, TRUE, &mii);
}
-
-
-
#define LANGUAGE_MENU_ID_BASE 10
#define WORD_MENU_ID_BASE 100
#define AUTOREPLACE_MENU_ID_BASE 50
@@ -1266,8 +1124,7 @@ void AddMenuForWord(Dialog *dlg, TCHAR *word, CHARRANGE &pos, HMENU hMenu, BOOL
Suggestions &suggestions = data.suggestions;
- if (in_submenu)
- {
+ if (in_submenu) {
data.hMeSubMenu = CreatePopupMenu();
AppendSubmenu(hMenu, data.hMeSubMenu, word);
hMenu = data.hMeSubMenu;
@@ -1276,8 +1133,7 @@ void AddMenuForWord(Dialog *dlg, TCHAR *word, CHARRANGE &pos, HMENU hMenu, BOOL
data.hReplaceSubMenu = CreatePopupMenu();
InsertMenu(data.hReplaceSubMenu, 0, MF_BYPOSITION, base + AUTOREPLACE_MENU_ID_BASE + suggestions.count, TranslateT("Other..."));
- if (suggestions.count > 0)
- {
+ if (suggestions.count > 0) {
InsertMenu(data.hReplaceSubMenu, 0, MF_BYPOSITION | MF_SEPARATOR, 0, 0);
for (int i = suggestions.count-1; i >= 0; i--)
InsertMenu(data.hReplaceSubMenu, 0, MF_BYPOSITION, base + AUTOREPLACE_MENU_ID_BASE + i, suggestions.words[i]);
@@ -1288,16 +1144,13 @@ void AddMenuForWord(Dialog *dlg, TCHAR *word, CHARRANGE &pos, HMENU hMenu, BOOL
InsertMenu(hMenu, 0, MF_BYPOSITION, base + suggestions.count + 1, TranslateT("Ignore all"));
InsertMenu(hMenu, 0, MF_BYPOSITION, base + suggestions.count, TranslateT("Add to dictionary"));
- if (suggestions.count > 0)
- {
+ if (suggestions.count > 0) {
HMENU hSubMenu;
- if (opts.cascade_corrections)
- {
+ if (opts.cascade_corrections) {
hSubMenu = data.hCorrectSubMenu = CreatePopupMenu();
AppendSubmenu(hMenu, hSubMenu, TranslateT("Corrections"));
}
- else
- {
+ else {
InsertMenu(hMenu, 0, MF_BYPOSITION | MF_SEPARATOR, 0, 0);
hSubMenu = hMenu;
}
@@ -1306,8 +1159,7 @@ void AddMenuForWord(Dialog *dlg, TCHAR *word, CHARRANGE &pos, HMENU hMenu, BOOL
InsertMenu(hSubMenu, 0, MF_BYPOSITION, base + i, suggestions.words[i]);
}
- if (!in_submenu && opts.show_wrong_word)
- {
+ if (!in_submenu && opts.show_wrong_word) {
InsertMenu(hMenu, 0, MF_BYPOSITION | MF_SEPARATOR, 0, 0);
TCHAR text[128];
@@ -1316,7 +1168,6 @@ void AddMenuForWord(Dialog *dlg, TCHAR *word, CHARRANGE &pos, HMENU hMenu, BOOL
}
}
-
struct FoundWrongWordParam {
Dialog *dlg;
int count;
@@ -1334,8 +1185,7 @@ void FoundWrongWord(TCHAR *word, CHARRANGE pos, void *param)
void AddItemsToMenu(Dialog *dlg, HMENU hMenu, POINT pt, HWND hwndOwner)
{
FreePopupData(dlg);
- if (opts.use_flags)
- {
+ if (opts.use_flags) {
dlg->hwnd_menu_owner = hwndOwner;
menus[hwndOwner] = dlg;
}
@@ -1346,8 +1196,7 @@ void AddItemsToMenu(Dialog *dlg, HMENU hMenu, POINT pt, HWND hwndOwner)
if (GetMenuItemCount(hMenu) > 0)
InsertMenu(hMenu, 0, MF_BYPOSITION | MF_SEPARATOR, 0, 0);
- if (languages.getCount() > 0 && dlg->enabled)
- {
+ if (languages.getCount() > 0 && dlg->enabled) {
dlg->hLanguageSubMenu = CreatePopupMenu();
if (dlg->hwnd_menu_owner != NULL)
@@ -1355,10 +1204,8 @@ void AddItemsToMenu(Dialog *dlg, HMENU hMenu, POINT pt, HWND hwndOwner)
// First add languages
for (int i = 0; i < languages.getCount(); i++)
- {
AppendMenu(dlg->hLanguageSubMenu, MF_STRING | (languages[i] == dlg->lang ? MF_CHECKED : 0),
LANGUAGE_MENU_ID_BASE + i, languages[i]->full_name);
- }
AppendSubmenu(hMenu, dlg->hLanguageSubMenu, TranslateT("Language"));
}
@@ -1367,10 +1214,8 @@ void AddItemsToMenu(Dialog *dlg, HMENU hMenu, POINT pt, HWND hwndOwner)
CheckMenuItem(hMenu, 1, MF_BYCOMMAND | (dlg->enabled ? MF_CHECKED : MF_UNCHECKED));
// Get text
- if (dlg->lang != NULL && dlg->enabled)
- {
- if (opts.show_all_corrections)
- {
+ if (dlg->lang != NULL && dlg->enabled) {
+ if (opts.show_all_corrections) {
dlg->hWrongWordsSubMenu = CreatePopupMenu();
FoundWrongWordParam p = { dlg, 0 };
@@ -1379,14 +1224,11 @@ void AddItemsToMenu(Dialog *dlg, HMENU hMenu, POINT pt, HWND hwndOwner)
if (p.count > 0)
AppendSubmenu(hMenu, dlg->hWrongWordsSubMenu, TranslateT("Wrong words"));
}
- else
- {
+ else {
CHARRANGE sel;
TCHAR *word = GetWordUnderPoint(dlg, pt, sel);
- if (word != NULL && !dlg->lang->spell(word))
- {
+ if (word != NULL && !dlg->lang->spell(word)) {
InsertMenu(hMenu, 0, MF_BYPOSITION | MF_SEPARATOR, 0, 0);
-
AddMenuForWord(dlg, word, sel, hMenu, FALSE, WORD_MENU_ID_BASE);
}
}
@@ -1413,14 +1255,11 @@ BOOL HandleMenuSelection(Dialog *dlg, POINT pt, unsigned selection)
{
BOOL ret = FALSE;
- if (selection == 1)
- {
+ if (selection == 1) {
ToggleEnabled(dlg);
-
ret = TRUE;
}
- else if (selection >= LANGUAGE_MENU_ID_BASE && selection < LANGUAGE_MENU_ID_BASE + (unsigned) languages.getCount())
- {
+ else if (selection >= LANGUAGE_MENU_ID_BASE && selection < LANGUAGE_MENU_ID_BASE + (unsigned) languages.getCount()) {
SetNoUnderline(dlg);
if (dlg->hContact == NULL)
@@ -1446,38 +1285,27 @@ BOOL HandleMenuSelection(Dialog *dlg, POINT pt, unsigned selection)
pos--; // 0 based
WrongWordPopupMenuData &data = (*dlg->wrong_words)[pos];
- if (selection < data.suggestions.count)
- {
+ if (selection < data.suggestions.count) {
// TODO Assert that text hasn't changed
ReplaceWord(dlg, data.pos, data.suggestions.words[selection]);
-
ret = TRUE;
}
- else if (selection == data.suggestions.count)
- {
+ else if (selection == data.suggestions.count) {
dlg->lang->addWord(data.word);
-
ret = TRUE;
}
- else if (selection == data.suggestions.count + 1)
- {
+ else if (selection == data.suggestions.count + 1) {
dlg->lang->ignoreWord(data.word);
-
ret = TRUE;
}
- else if (selection >= AUTOREPLACE_MENU_ID_BASE
- && selection < AUTOREPLACE_MENU_ID_BASE + data.suggestions.count + 1)
- {
+ else if (selection >= AUTOREPLACE_MENU_ID_BASE && selection < AUTOREPLACE_MENU_ID_BASE + data.suggestions.count + 1) {
selection -= AUTOREPLACE_MENU_ID_BASE;
-
- if (selection == data.suggestions.count)
- {
+ if (selection == data.suggestions.count) {
ShowAutoReplaceDialog(dlg->hwnd_owner != NULL ? dlg->hwnd_owner : dlg->hwnd, FALSE,
dlg->lang, data.word, NULL, FALSE,
TRUE, &AddWordToDictCallback, dlg->hwnd);
}
- else
- {
+ else {
// TODO Assert that text hasn't changed
ReplaceWord(dlg, data.pos, data.suggestions.words[selection]);
dlg->lang->autoReplace->add(data.word, data.suggestions.words[selection]);
@@ -1486,8 +1314,7 @@ BOOL HandleMenuSelection(Dialog *dlg, POINT pt, unsigned selection)
}
}
- if (ret)
- {
+ if (ret) {
KillTimer(dlg->hwnd, TIMER_ID);
SetTimer(dlg->hwnd, TIMER_ID, 100, NULL);
@@ -1495,16 +1322,13 @@ BOOL HandleMenuSelection(Dialog *dlg, POINT pt, unsigned selection)
}
FreePopupData(dlg);
-
return ret;
}
-
int MsgWindowPopup(WPARAM wParam, LPARAM lParam)
{
MessageWindowPopupData *mwpd = (MessageWindowPopupData *) lParam;
- if (mwpd == NULL || mwpd->cbSize < sizeof(MessageWindowPopupData)
- || mwpd->uFlags != MSG_WINDOWPOPUP_INPUT)
+ if (mwpd == NULL || mwpd->cbSize < sizeof(MessageWindowPopupData) || mwpd->uFlags != MSG_WINDOWPOPUP_INPUT)
return 0;
DialogMapType::iterator dlgit = dialogs.find(mwpd->hwnd);
@@ -1517,17 +1341,13 @@ int MsgWindowPopup(WPARAM wParam, LPARAM lParam)
ScreenToClient(dlg->hwnd, &pt);
if (mwpd->uType == MSG_WINDOWPOPUP_SHOWING)
- {
AddItemsToMenu(dlg, mwpd->hMenu, pt, dlg->hwnd_owner);
- }
else if (mwpd->uType == MSG_WINDOWPOPUP_SELECTED)
- {
HandleMenuSelection(dlg, pt, mwpd->selection);
- }
+
return 0;
}
-
INT_PTR ShowPopupMenuService(WPARAM wParam, LPARAM lParam)
{
SPELLCHECKER_POPUPMENU *scp = (SPELLCHECKER_POPUPMENU *) wParam;
@@ -1537,7 +1357,6 @@ INT_PTR ShowPopupMenuService(WPARAM wParam, LPARAM lParam)
return ShowPopupMenu(scp->hwnd, scp->hMenu, scp->pt, scp->hwndOwner == NULL ? scp->hwnd : scp->hwndOwner);
}
-
int ShowPopupMenu(HWND hwnd, HMENU hMenu, POINT pt, HWND hwndOwner)
{
DialogMapType::iterator dlgit = dialogs.find(hwnd);
@@ -1546,8 +1365,7 @@ int ShowPopupMenu(HWND hwnd, HMENU hMenu, POINT pt, HWND hwndOwner)
Dialog *dlg = dlgit->second;
- if (pt.x == 0xFFFF && pt.y == 0xFFFF)
- {
+ if (pt.x == 0xFFFF && pt.y == 0xFFFF) {
CHARRANGE sel;
SendMessage(hwnd, EM_EXGETSEL, 0, (LPARAM) &sel);
@@ -1578,7 +1396,6 @@ int ShowPopupMenu(HWND hwnd, HMENU hMenu, POINT pt, HWND hwndOwner)
return selection;
}
-
int MsgWindowEvent(WPARAM wParam, LPARAM lParam)
{
MessageWindowEventData *event = (MessageWindowEventData *)lParam;
@@ -1589,13 +1406,9 @@ int MsgWindowEvent(WPARAM wParam, LPARAM lParam)
return 0;
if (event->uType == MSG_WINDOW_EVT_OPEN)
- {
AddContactTextBox(event->hContact, event->hwndInput, "DefaultSRMM", TRUE, event->hwndWindow);
- }
else if (event->uType == MSG_WINDOW_EVT_CLOSING)
- {
RemoveContactTextBox(event->hwndInput);
- }
return 0;
}
@@ -1614,33 +1427,27 @@ int IconPressed(WPARAM wParam, LPARAM lParam)
// Find the dialog
HWND hwnd = NULL;
Dialog *dlg;
- for(DialogMapType::iterator it = dialogs.begin(); it != dialogs.end(); it++)
- {
+ for(DialogMapType::iterator it = dialogs.begin(); it != dialogs.end(); it++) {
dlg = it->second;
- if (dlg->srmm && dlg->hContact == hContact)
- {
+ if (dlg->srmm && dlg->hContact == hContact) {
hwnd = it->first;
break;
}
}
- if (hwnd == NULL)
- {
+ if (hwnd == NULL) {
NotifyWrongSRMM();
return 0;
}
- if ((sicd->flags & MBCF_RIGHTBUTTON) == 0)
- {
+ if ((sicd->flags & MBCF_RIGHTBUTTON) == 0) {
FreePopupData(dlg);
// Show the menu
HMENU hMenu = CreatePopupMenu();
- if (languages.getCount() > 0)
- {
- if (opts.use_flags)
- {
+ if (languages.getCount() > 0) {
+ if (opts.use_flags) {
menus[dlg->hwnd] = dlg;
dlg->hwnd_menu_owner = dlg->hwnd;
dlg->old_menu_proc = (WNDPROC) SetWindowLongPtr(dlg->hwnd_menu_owner, GWLP_WNDPROC, (LONG_PTR) MenuWndProc);
@@ -1648,10 +1455,8 @@ int IconPressed(WPARAM wParam, LPARAM lParam)
// First add languages
for (int i = 0; i < languages.getCount(); i++)
- {
AppendMenu(hMenu, MF_STRING | (languages[i] == dlg->lang ? MF_CHECKED : 0),
LANGUAGE_MENU_ID_BASE + i, languages[i]->full_name);
- }
InsertMenu(hMenu, 0, MF_BYPOSITION | MF_SEPARATOR, 0, 0);
}
@@ -1660,15 +1465,11 @@ int IconPressed(WPARAM wParam, LPARAM lParam)
CheckMenuItem(hMenu, 1, MF_BYCOMMAND | (dlg->enabled ? MF_CHECKED : MF_UNCHECKED));
// Show menu
- int selection = TrackPopupMenu(hMenu, TPM_RETURNCMD, sicd->clickLocation.x, sicd->clickLocation.y, 0,
- dlg->hwnd, NULL);
-
+ int selection = TrackPopupMenu(hMenu, TPM_RETURNCMD, sicd->clickLocation.x, sicd->clickLocation.y, 0, dlg->hwnd, NULL);
HandleMenuSelection(dlg, sicd->clickLocation, selection);
-
DestroyMenu(hMenu);
}
- else
- {
+ else {
// Enable / disable
HandleMenuSelection(dlg, sicd->clickLocation, 1);
}
@@ -1676,7 +1477,6 @@ int IconPressed(WPARAM wParam, LPARAM lParam)
return 0;
}
-
LRESULT CALLBACK MenuWndProc(HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam)
{
DialogMapType::iterator dlgit = menus.find(hwnd);
@@ -1691,8 +1491,7 @@ LRESULT CALLBACK MenuWndProc(HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam)
HMENU hMenu = (HMENU) wParam;
int count = GetMenuItemCount(hMenu);
- for(int i = 0; i < count; i++)
- {
+ for(int i = 0; i < count; i++) {
unsigned id = GetMenuItemID(hMenu, i);
if (id < LANGUAGE_MENU_ID_BASE || id >= LANGUAGE_MENU_ID_BASE + (unsigned) languages.getCount())
continue;
@@ -1817,8 +1616,7 @@ TCHAR *lstrtrim(TCHAR *str)
int i;
for(i = len - 1; i >= 0 && (str[i] == ' ' || str[i] == '\t'); --i) ;
- if (i < len - 1)
- {
+ if (i < len - 1) {
++i;
str[i] = _T('\0');
len = i;
@@ -1833,7 +1631,6 @@ TCHAR *lstrtrim(TCHAR *str)
BOOL lstreq(TCHAR *a, TCHAR *b, size_t len)
{
-
a = CharLower(_tcsdup(a));
b = CharLower(_tcsdup(b));
BOOL ret;
@@ -1844,5 +1641,4 @@ BOOL lstreq(TCHAR *a, TCHAR *b, size_t len)
free(a);
free(b);
return ret;
-
}
diff --git a/plugins/Utils/mir_options.cpp b/plugins/Utils/mir_options.cpp
index 8a31f3c1ee..b6ccd76eda 100644
--- a/plugins/Utils/mir_options.cpp
+++ b/plugins/Utils/mir_options.cpp
@@ -42,7 +42,7 @@ static TCHAR* MyDBGetContactSettingTString(HANDLE hContact, char* module, char*
out[0] = _T('\0');
- if ( !DBGetContactSettingTString(hContact, module, setting, &dbv))
+ if ( !db_get_ts(hContact, module, setting, &dbv))
{
lstrcpyn(out, dbv.ptszVal, (int)len);
db_free(&dbv);