minus critical section

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

1 files changed, 147 insertions, 143 deletions

diff --git a/plugins/Ping/src/options.cpp b/plugins/Ping/src/options.cpp
index 8d6ef33cc4..b2f698a9cf 100644
--- a/plugins/Ping/src/options.cpp
+++ b/plugins/Ping/src/options.cpp
@@ -5,11 +5,11 @@ PingOptions options;
 // main ping options 

 static INT_PTR CALLBACK DlgProcOpts(HWND hwndDlg, UINT msg, WPARAM wParam, LPARAM lParam)

 {

-	switch ( msg ) {

+	switch (msg) {

 	case WM_INITDIALOG:

-		TranslateDialogDefault( hwndDlg );

+		TranslateDialogDefault(hwndDlg);

 

-		if(ServiceExists(MS_CLIST_FRAMES_ADDFRAME))

+		if (ServiceExists(MS_CLIST_FRAMES_ADDFRAME))

 			EnableWindow(GetDlgItem(hwndDlg, IDC_CHK_ATTACH), FALSE);

 

 		CheckDlgButton(hwndDlg, IDC_CHK_ATTACH, options.attach_to_clist ? BST_CHECKED : BST_UNCHECKED);

@@ -31,37 +31,37 @@ static INT_PTR CALLBACK DlgProcOpts(HWND hwndDlg, UINT msg, WPARAM wParam, LPARA
 		SetDlgItemInt(hwndDlg, IDC_RPT, options.retries, FALSE);

 

 		SetDlgItemText(hwndDlg, IDC_ED_FILENAME, options.log_filename);

-		if(!options.logging) {

+		if (!options.logging) {

 			EnableWindow(GetDlgItem(hwndDlg, IDC_ED_FILENAME), FALSE);

 			EnableWindow(GetDlgItem(hwndDlg, IDC_BTN_LOGBROWSE), FALSE);

 			EnableWindow(GetDlgItem(hwndDlg, IDC_CHK_LOGCSV), FALSE);

 		}

 

-		if( !ServiceExists( MS_POPUP_ADDPOPUPT )) {

+		if (!ServiceExists(MS_POPUP_ADDPOPUPT)) {

 			EnableWindow(GetDlgItem(hwndDlg, IDC_CHECKPOPUP), FALSE);

 			EnableWindow(GetDlgItem(hwndDlg, IDC_CHECKPOPUP2), FALSE);

 			EnableWindow(GetDlgItem(hwndDlg, IDC_CHK_BLOCK), FALSE);

-		} 

+		}

 		return TRUE;

 	case WM_COMMAND:

-		if ( HIWORD( wParam ) == EN_CHANGE && ( HWND )lParam == GetFocus()) {

-			switch( LOWORD( wParam )) {

+		if (HIWORD(wParam) == EN_CHANGE && (HWND)lParam == GetFocus()) {

+			switch (LOWORD(wParam)) {

 			case IDC_PPM:

 			case IDC_PT:

 			case IDC_ED_FILENAME:

 			case IDC_RPT:

 				SendMessage(GetParent(hwndDlg), PSM_CHANGED, 0, 0);

-			}	

+			}

 			break;

 		}

 

-		if (HIWORD( wParam ) == CBN_SELCHANGE) {

+		if (HIWORD(wParam) == CBN_SELCHANGE) {

 			SendMessage(GetParent(hwndDlg), PSM_CHANGED, 0, 0);

 			break;

 		}

 

-		if ( HIWORD( wParam ) == BN_CLICKED ) {

-			switch( LOWORD( wParam )) {

+		if (HIWORD(wParam) == BN_CLICKED) {

+			switch (LOWORD(wParam)) {

 			case IDC_CHK_LOG:

 				EnableWindow(GetDlgItem(hwndDlg, IDC_ED_FILENAME), IsDlgButtonChecked(hwndDlg, IDC_CHK_LOG));

 				EnableWindow(GetDlgItem(hwndDlg, IDC_BTN_LOGBROWSE), IsDlgButtonChecked(hwndDlg, IDC_CHK_LOG));

@@ -74,7 +74,7 @@ static INT_PTR CALLBACK DlgProcOpts(HWND hwndDlg, UINT msg, WPARAM wParam, LPARA
 			case IDC_CHK_MINMAX:

 			case IDC_CHK_NOTESTICON:

 			case IDC_CHK_ATTACH:

-				SendMessage( GetParent( hwndDlg ), PSM_CHANGED, 0, 0 );

+				SendMessage(GetParent(hwndDlg), PSM_CHANGED, 0, 0);

 				break;

 			case IDC_BTN_VIEWLOG:

 				CallService(PLUG "/ViewLogData", 0, 0);

@@ -83,7 +83,7 @@ static INT_PTR CALLBACK DlgProcOpts(HWND hwndDlg, UINT msg, WPARAM wParam, LPARA
 			{

 				TCHAR filter[MAX_PATH];

 				mir_sntprintf(filter, SIZEOF(filter), _T("%s%c*.txt%c%s%c*.*%c"), TranslateT("Text Files (*.txt)"), 0, 0, TranslateT("All Files"), 0, 0);

-				OPENFILENAME ofn = {0};

+				OPENFILENAME ofn = { 0 };

 				ofn.lStructSize = sizeof(ofn);

 				ofn.lpstrFile = options.log_filename;

 				ofn.hwndOwner = hwndDlg;

@@ -96,31 +96,31 @@ static INT_PTR CALLBACK DlgProcOpts(HWND hwndDlg, UINT msg, WPARAM wParam, LPARA
 				ofn.lpstrInitialDir = NULL;

 				ofn.Flags = OFN_PATHMUSTEXIST;

 

-				if(GetOpenFileName(&ofn) == TRUE) {

+				if (GetOpenFileName(&ofn) == TRUE) {

 					SetDlgItemText(hwndDlg, IDC_ED_FILENAME, ofn.lpstrFile);

-					SendMessage( GetParent( hwndDlg ), PSM_CHANGED, 0, 0 );

+					SendMessage(GetParent(hwndDlg), PSM_CHANGED, 0, 0);

 				}

 			}

-				break;

+			break;

 			}

 			break;

 		}

 		break;

 

 	case WM_NOTIFY:

-		if (((LPNMHDR)lParam)->code == UDN_DELTAPOS ) {

-			SendMessage( GetParent( hwndDlg ), PSM_CHANGED, 0, 0 );

-		} 

-		if (((LPNMHDR)lParam)->code == PSN_APPLY ) {

+		if (((LPNMHDR)lParam)->code == UDN_DELTAPOS) {

+			SendMessage(GetParent(hwndDlg), PSM_CHANGED, 0, 0);

+		}

+		if (((LPNMHDR)lParam)->code == PSN_APPLY) {

 

 			BOOL trans_success;

 

-			DWORD new_ping_period = GetDlgItemInt( hwndDlg, IDC_PPM, &trans_success, FALSE);

-			if(trans_success) {

+			DWORD new_ping_period = GetDlgItemInt(hwndDlg, IDC_PPM, &trans_success, FALSE);

+			if (trans_success) {

 				options.ping_period = new_ping_period;

 			}

-			DWORD new_ping_timeout = GetDlgItemInt( hwndDlg, IDC_PT, &trans_success, FALSE);

-			if(trans_success) {

+			DWORD new_ping_timeout = GetDlgItemInt(hwndDlg, IDC_PT, &trans_success, FALSE);

+			if (trans_success) {

 				options.ping_timeout = new_ping_timeout;

 			}

 			options.show_popup = IsDlgButtonChecked(hwndDlg, IDC_CHECKPOPUP) == BST_CHECKED;

@@ -141,7 +141,7 @@ static INT_PTR CALLBACK DlgProcOpts(HWND hwndDlg, UINT msg, WPARAM wParam, LPARA
 			}

 

 			bool new_attach = (IsDlgButtonChecked(hwndDlg, IDC_CHK_ATTACH) == BST_CHECKED);

-			if(!ServiceExists(MS_CLIST_FRAMES_ADDFRAME) && options.attach_to_clist != new_attach)

+			if (!ServiceExists(MS_CLIST_FRAMES_ADDFRAME) && options.attach_to_clist != new_attach)

 				AttachToClist(new_attach);

 

 			options.attach_to_clist = new_attach;

@@ -150,8 +150,8 @@ static INT_PTR CALLBACK DlgProcOpts(HWND hwndDlg, UINT msg, WPARAM wParam, LPARA
 

 			RefreshWindow(0, 0);

 

-			if(options.logging) CallService(PLUG "/Log", (WPARAM)_T("options changed"), 0);

-			if(hWakeEvent) SetEvent(hWakeEvent);

+			if (options.logging) CallService(PLUG "/Log", (WPARAM)_T("options changed"), 0);

+			if (hWakeEvent) SetEvent(hWakeEvent);

 			return TRUE;

 		}

 		break;

@@ -165,10 +165,10 @@ PINGADDRESS add_edit_addr;
 

 // host edit

 INT_PTR CALLBACK DlgProcDestEdit(HWND hwndDlg, UINT msg, WPARAM wParam, LPARAM lParam) {

-	switch ( msg ) {

-	case WM_INITDIALOG: 

+	switch (msg) {

+	case WM_INITDIALOG:

 		TranslateDialogDefault(hwndDlg);

-		for(int i = ID_STATUS_OFFLINE; i <= ID_STATUS_OUTTOLUNCH; i++) {

+		for (int i = ID_STATUS_OFFLINE; i <= ID_STATUS_OUTTOLUNCH; i++) {

 			INT_PTR ret = CallService(MS_CLIST_GETSTATUSMODEDESCRIPTION, (WPARAM)i, GSMDF_TCHAR);

 			SendDlgItemMessage(hwndDlg, IDC_COMBO_DESTSTAT, CB_INSERTSTRING, (WPARAM)-1, (LPARAM)ret);

 			SendDlgItemMessage(hwndDlg, IDC_COMBO_DESTSTAT2, CB_INSERTSTRING, (WPARAM)-1, (LPARAM)ret);

@@ -183,7 +183,7 @@ INT_PTR CALLBACK DlgProcDestEdit(HWND hwndDlg, UINT msg, WPARAM wParam, LPARAM l
 		SetDlgItemText(hwndDlg, IDC_ED_PARAMS, add_edit_addr.pszParams);

 

 		CheckDlgButton(hwndDlg, IDC_CHK_DESTTCP, add_edit_addr.port != -1 ? BST_CHECKED : BST_UNCHECKED);

-		if(add_edit_addr.port != -1) {

+		if (add_edit_addr.port != -1) {

 			EnableWindow(GetDlgItem(hwndDlg, IDC_ED_DESTPORT), TRUE);

 			SetDlgItemInt(hwndDlg, IDC_ED_DESTPORT, add_edit_addr.port, FALSE);

 		}

@@ -195,15 +195,16 @@ INT_PTR CALLBACK DlgProcDestEdit(HWND hwndDlg, UINT msg, WPARAM wParam, LPARAM l
 			int num_protocols = 0;

 			PROTOACCOUNT **pppDesc;

 

-			ProtoEnumAccounts(&num_protocols,&pppDesc);

-			for(int i = 0; i < num_protocols; i++)

+			ProtoEnumAccounts(&num_protocols, &pppDesc);

+			for (int i = 0; i < num_protocols; i++)

 				SendDlgItemMessage(hwndDlg, IDC_COMBO_DESTPROTO, CB_INSERTSTRING, (WPARAM)-1, (LPARAM)pppDesc[i]->tszAccountName);

-				

+

 		}

 

-		if(add_edit_addr.pszProto[0] == '\0') {

+		if (add_edit_addr.pszProto[0] == '\0') {

 			SendDlgItemMessage(hwndDlg, IDC_COMBO_DESTPROTO, CB_SETCURSEL, 0, 0);

-		} else {

+		}

+		else {

 			SendDlgItemMessage(hwndDlg, IDC_COMBO_DESTPROTO, CB_SELECTSTRING, 0, (LPARAM)add_edit_addr.pszProto);

 			EnableWindow(GetDlgItem(hwndDlg, IDC_COMBO_DESTSTAT), TRUE);

 			SendDlgItemMessage(hwndDlg, IDC_COMBO_DESTSTAT, CB_SETCURSEL, (WPARAM)(add_edit_addr.set_status - ID_STATUS_OFFLINE), 0);

@@ -216,17 +217,17 @@ INT_PTR CALLBACK DlgProcDestEdit(HWND hwndDlg, UINT msg, WPARAM wParam, LPARAM l
 		SetFocus(GetDlgItem(hwndDlg, IDC_ED_DESTLAB));

 		return FALSE;

 	case WM_COMMAND:

-		if (HIWORD( wParam ) == LBN_SELCHANGE && LOWORD(wParam) == IDC_COMBO_DESTPROTO) {

+		if (HIWORD(wParam) == LBN_SELCHANGE && LOWORD(wParam) == IDC_COMBO_DESTPROTO) {

 			int sel = SendDlgItemMessage(hwndDlg, IDC_COMBO_DESTPROTO, CB_GETCURSEL, 0, 0);

-			if(sel != CB_ERR) {

+			if (sel != CB_ERR) {

 				EnableWindow(GetDlgItem(hwndDlg, IDC_COMBO_DESTSTAT), sel != 0);

 				EnableWindow(GetDlgItem(hwndDlg, IDC_COMBO_DESTSTAT2), sel != 0);

 			}

 		}

 

-		if ( HIWORD( wParam ) == BN_CLICKED )

+		if (HIWORD(wParam) == BN_CLICKED)

 		{

-			switch( LOWORD( wParam ))

+			switch (LOWORD(wParam))

 			{

 			case IDC_CHK_DESTTCP:

 				EnableWindow(GetDlgItem(hwndDlg, IDC_ED_DESTPORT), IsDlgButtonChecked(hwndDlg, IDC_CHK_DESTTCP));

@@ -237,30 +238,32 @@ INT_PTR CALLBACK DlgProcDestEdit(HWND hwndDlg, UINT msg, WPARAM wParam, LPARAM l
 				GetDlgItemText(hwndDlg, IDC_ED_COMMAND, add_edit_addr.pszCommand, SIZEOF(add_edit_addr.pszCommand));

 				GetDlgItemText(hwndDlg, IDC_ED_PARAMS, add_edit_addr.pszParams, SIZEOF(add_edit_addr.pszParams));

 

-				if(SendDlgItemMessage(hwndDlg, IDC_COMBO_DESTPROTO, CB_GETCURSEL, 0, 0) != -1)

+				if (SendDlgItemMessage(hwndDlg, IDC_COMBO_DESTPROTO, CB_GETCURSEL, 0, 0) != -1)

 				{

 					GetDlgItemTextA(hwndDlg, IDC_COMBO_DESTPROTO, add_edit_addr.pszProto, SIZEOF(add_edit_addr.pszProto));

-					if(!strcmp(add_edit_addr.pszProto, Translate("<none>")))

+					if (!strcmp(add_edit_addr.pszProto, Translate("<none>")))

 						add_edit_addr.pszProto[0] = '\0';

 					else {

 						int sel = SendDlgItemMessage(hwndDlg, IDC_COMBO_DESTSTAT, CB_GETCURSEL, 0, 0);

-						if(sel != CB_ERR)

+						if (sel != CB_ERR)

 							add_edit_addr.set_status = ID_STATUS_OFFLINE + sel;

 

 						sel = SendDlgItemMessage(hwndDlg, IDC_COMBO_DESTSTAT2, CB_GETCURSEL, 0, 0);

-						if(sel != CB_ERR)

+						if (sel != CB_ERR)

 							add_edit_addr.get_status = ID_STATUS_OFFLINE + sel;

 					}

-				} else

+				}

+				else

 					add_edit_addr.pszProto[0] = '\0';

 

 				if (IsDlgButtonChecked(hwndDlg, IDC_CHK_DESTTCP))

 				{

 					BOOL tr;

 					int port = GetDlgItemInt(hwndDlg, IDC_ED_DESTPORT, &tr, FALSE);

-					if(tr) add_edit_addr.port = port;

+					if (tr) add_edit_addr.port = port;

 					else add_edit_addr.port = -1;

-				} else

+				}

+				else

 					add_edit_addr.port = -1;

 

 				EndDialog(hwndDlg, IDOK);

@@ -292,26 +295,27 @@ bool Edit(HWND hwnd, PINGADDRESS &addr)
 // ping hosts list window

 static INT_PTR CALLBACK DlgProcOpts2(HWND hwndDlg, UINT msg, WPARAM wParam, LPARAM lParam)

 {

-	switch ( msg ) {

-	case WM_INITDIALOG: 

-			TranslateDialogDefault( hwndDlg );

+	switch (msg) {

+	case WM_INITDIALOG:

+	{

+		TranslateDialogDefault(hwndDlg);

 

-			Lock(&data_list_cs, "init options dialog");

-			temp_list = data_list;

-			Unlock(&data_list_cs);

+		mir_cslock lck(data_list_cs);

+		temp_list = data_list;

 

-			for (pinglist_it i = temp_list.begin(); i != temp_list.end(); ++i)

-			{

-				int index = SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_INSERTSTRING, (WPARAM)-1, (LPARAM)i->pszLabel);

-				SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_SETITEMDATA, index, (LPARAM)&(*i));

-			}

-		return TRUE;

+		for (pinglist_it i = temp_list.begin(); i != temp_list.end(); ++i)

+		{

+			int index = SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_INSERTSTRING, (WPARAM)-1, (LPARAM)i->pszLabel);

+			SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_SETITEMDATA, index, (LPARAM)&(*i));

+		}

+	}

+	return TRUE;

 

 	case WM_COMMAND:

-		if (HIWORD( wParam ) == LBN_SELCHANGE && LOWORD(wParam) == IDC_LST_DEST)

+		if (HIWORD(wParam) == LBN_SELCHANGE && LOWORD(wParam) == IDC_LST_DEST)

 		{

 			int sel = SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_GETCURSEL, 0, 0);

-			if(sel != LB_ERR)

+			if (sel != LB_ERR)

 			{

 				EnableWindow(GetDlgItem(hwndDlg, IDC_BTN_DESTREM), TRUE);

 				EnableWindow(GetDlgItem(hwndDlg, IDC_BTN_DESTEDIT), TRUE);

@@ -322,9 +326,9 @@ static INT_PTR CALLBACK DlgProcOpts2(HWND hwndDlg, UINT msg, WPARAM wParam, LPAR
 			}

 		}

 

-		if ( HIWORD( wParam ) == BN_CLICKED )

+		if (HIWORD(wParam) == BN_CLICKED)

 		{

-			switch( LOWORD( wParam ))

+			switch (LOWORD(wParam))

 			{

 			case IDC_BTN_DESTEDIT:

 			{

@@ -347,14 +351,14 @@ static INT_PTR CALLBACK DlgProcOpts2(HWND hwndDlg, UINT msg, WPARAM wParam, LPAR
 						int count = SendDlgItemMessage(hwndDlg, IDC_BTN_DESTDOWN, LB_GETCOUNT, 0, 0);

 						EnableWindow(GetDlgItem(hwndDlg, IDC_BTN_DESTDOWN), (sel < count - 1));

 

-						SendMessage( GetParent( hwndDlg ), PSM_CHANGED, 0, 0 );

+						SendMessage(GetParent(hwndDlg), PSM_CHANGED, 0, 0);

 					}

 				}

 			}

-				break;

+			break;

 			case IDC_BTN_DESTADD:

 

-				memset(&add_edit_addr,0,sizeof(add_edit_addr));

+				memset(&add_edit_addr, 0, sizeof(add_edit_addr));

 				add_edit_addr.cbSize = sizeof(add_edit_addr);

 				add_edit_addr.port = -1;

 				add_edit_addr.set_status = ID_STATUS_ONLINE;

@@ -363,7 +367,7 @@ static INT_PTR CALLBACK DlgProcOpts2(HWND hwndDlg, UINT msg, WPARAM wParam, LPAR
 				add_edit_addr.item_id = 0;

 				add_edit_addr.index = (int)temp_list.size();

 

-				if(DialogBox(hInst, MAKEINTRESOURCE(IDD_DIALOG3), hwndDlg, DlgProcDestEdit) == IDOK)

+				if (DialogBox(hInst, MAKEINTRESOURCE(IDD_DIALOG3), hwndDlg, DlgProcDestEdit) == IDOK)

 				{

 					temp_list.push_back(add_edit_addr);

 

@@ -379,14 +383,14 @@ static INT_PTR CALLBACK DlgProcOpts2(HWND hwndDlg, UINT msg, WPARAM wParam, LPAR
 					int count = SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_GETCOUNT, 0, 0);

 					EnableWindow(GetDlgItem(hwndDlg, IDC_BTN_DESTDOWN), (sel < count - 1));

 

-					SendMessage( GetParent( hwndDlg ), PSM_CHANGED, 0, 0 );

+					SendMessage(GetParent(hwndDlg), PSM_CHANGED, 0, 0);

 				}

 

 				break;

 			case IDC_BTN_DESTREM:

 			{

 				int sel = SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_GETCURSEL, 0, 0);

-				if(sel != LB_ERR) {

+				if (sel != LB_ERR) {

 					PINGADDRESS *item = (PINGADDRESS *)SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_GETITEMDATA, sel, 0);

 					SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_DELETESTRING, (WPARAM)sel, 0);

 					temp_list.remove(*item);

@@ -397,93 +401,93 @@ static INT_PTR CALLBACK DlgProcOpts2(HWND hwndDlg, UINT msg, WPARAM wParam, LPAR
 				EnableWindow(GetDlgItem(hwndDlg, IDC_BTN_DESTUP), FALSE);

 				EnableWindow(GetDlgItem(hwndDlg, IDC_BTN_DESTDOWN), FALSE);

 

-				SendMessage( GetParent( hwndDlg ), PSM_CHANGED, 0, 0 );

+				SendMessage(GetParent(hwndDlg), PSM_CHANGED, 0, 0);

 				RefreshWindow(0, 0);

 				break;

 			}

 			case IDC_BTN_DESTDOWN:

-				{

-					int sel2 = SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_GETCURSEL, 0, 0);

-					if(sel2 != LB_ERR) {

-						PINGADDRESS *item = (PINGADDRESS *)SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_GETITEMDATA, sel2, 0),

-							*item2 = (PINGADDRESS *)SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_GETITEMDATA, sel2 + 1, 0);

-						if(item && item2)

-						{

-							add_edit_addr = *item;

-							*item = *item2;

-							*item2 = add_edit_addr;

-

-							// keep indexes the same, as they're used for sorting the binary tree

-							int index = item->index, index2 = item2->index;						

-							item->index = index2;

-							item2->index = index;

-

-							SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_DELETESTRING, (WPARAM)sel2, 0);

-							SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_INSERTSTRING, (WPARAM)sel2, (LPARAM)item->pszLabel);

-							SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_SETITEMDATA, (WPARAM)sel2, (LPARAM)item);

-							SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_DELETESTRING, (WPARAM)(sel2 + 1), 0);

-							SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_INSERTSTRING, (WPARAM)(sel2 + 1), (LPARAM)item2->pszLabel);

-							SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_SETITEMDATA, (WPARAM)(sel2 + 1), (LPARAM)item2);

-							SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_SETCURSEL, (WPARAM)(sel2 + 1), 0);

-

-							EnableWindow(GetDlgItem(hwndDlg, IDC_BTN_DESTUP), (sel2 + 1 > 0));

-							int count = SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_GETCOUNT, 0, 0);

-							EnableWindow(GetDlgItem(hwndDlg, IDC_BTN_DESTDOWN), (sel2 + 1 < count - 1));

-

-							SendMessage( GetParent( hwndDlg ), PSM_CHANGED, 0, 0 );

-						}

+			{

+				int sel2 = SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_GETCURSEL, 0, 0);

+				if (sel2 != LB_ERR) {

+					PINGADDRESS *item = (PINGADDRESS *)SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_GETITEMDATA, sel2, 0),

+						*item2 = (PINGADDRESS *)SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_GETITEMDATA, sel2 + 1, 0);

+					if (item && item2)

+					{

+						add_edit_addr = *item;

+						*item = *item2;

+						*item2 = add_edit_addr;

+

+						// keep indexes the same, as they're used for sorting the binary tree

+						int index = item->index, index2 = item2->index;

+						item->index = index2;

+						item2->index = index;

+

+						SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_DELETESTRING, (WPARAM)sel2, 0);

+						SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_INSERTSTRING, (WPARAM)sel2, (LPARAM)item->pszLabel);

+						SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_SETITEMDATA, (WPARAM)sel2, (LPARAM)item);

+						SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_DELETESTRING, (WPARAM)(sel2 + 1), 0);

+						SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_INSERTSTRING, (WPARAM)(sel2 + 1), (LPARAM)item2->pszLabel);

+						SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_SETITEMDATA, (WPARAM)(sel2 + 1), (LPARAM)item2);

+						SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_SETCURSEL, (WPARAM)(sel2 + 1), 0);

+

+						EnableWindow(GetDlgItem(hwndDlg, IDC_BTN_DESTUP), (sel2 + 1 > 0));

+						int count = SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_GETCOUNT, 0, 0);

+						EnableWindow(GetDlgItem(hwndDlg, IDC_BTN_DESTDOWN), (sel2 + 1 < count - 1));

+

+						SendMessage(GetParent(hwndDlg), PSM_CHANGED, 0, 0);

 					}

 				}

-				break;

+			}

+			break;

 			case IDC_BTN_DESTUP:

-				{

-					int sel2 = SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_GETCURSEL, 0, 0);

-					if(sel2 != LB_ERR) {

-						PINGADDRESS *item = (PINGADDRESS *)SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_GETITEMDATA, sel2, 0),

-							*item2 = (PINGADDRESS *)SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_GETITEMDATA, sel2 - 1, 0);

-

-						if (item && item2)

-						{

-							add_edit_addr = *item;

-							*item = *item2;

-							*item2 = add_edit_addr;

-

-							// keep indexes the same, as they're used for sorting the binary tree

-							int index = item->index, index2 = item2->index;						

-							item->index = index2;

-							item2->index = index;

-

-							SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_DELETESTRING, (WPARAM)sel2, 0);

-							SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_INSERTSTRING, (WPARAM)sel2, (LPARAM)item->pszLabel);

-							SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_SETITEMDATA, (WPARAM)sel2, (LPARAM)item);

-

-							SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_DELETESTRING, (WPARAM)(sel2 - 1), 0);

-							SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_INSERTSTRING, (WPARAM)(sel2 - 1), (LPARAM)item2->pszLabel);

-							SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_SETITEMDATA, (WPARAM)(sel2 - 1), (LPARAM)item2);

-

-							SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_SETCURSEL, (WPARAM)(sel2 - 1), 0);

-

-							EnableWindow(GetDlgItem(hwndDlg, IDC_BTN_DESTUP), (sel2 - 1 > 0));

-							int count = SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_GETCOUNT, 0, 0);

-							EnableWindow(GetDlgItem(hwndDlg, IDC_BTN_DESTDOWN), (sel2 - 1 < count - 1));

-

-							SendMessage( GetParent( hwndDlg ), PSM_CHANGED, 0, 0 );

-						}

+			{

+				int sel2 = SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_GETCURSEL, 0, 0);

+				if (sel2 != LB_ERR) {

+					PINGADDRESS *item = (PINGADDRESS *)SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_GETITEMDATA, sel2, 0),

+						*item2 = (PINGADDRESS *)SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_GETITEMDATA, sel2 - 1, 0);

+

+					if (item && item2)

+					{

+						add_edit_addr = *item;

+						*item = *item2;

+						*item2 = add_edit_addr;

+

+						// keep indexes the same, as they're used for sorting the binary tree

+						int index = item->index, index2 = item2->index;

+						item->index = index2;

+						item2->index = index;

+

+						SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_DELETESTRING, (WPARAM)sel2, 0);

+						SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_INSERTSTRING, (WPARAM)sel2, (LPARAM)item->pszLabel);

+						SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_SETITEMDATA, (WPARAM)sel2, (LPARAM)item);

+

+						SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_DELETESTRING, (WPARAM)(sel2 - 1), 0);

+						SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_INSERTSTRING, (WPARAM)(sel2 - 1), (LPARAM)item2->pszLabel);

+						SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_SETITEMDATA, (WPARAM)(sel2 - 1), (LPARAM)item2);

+

+						SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_SETCURSEL, (WPARAM)(sel2 - 1), 0);

+

+						EnableWindow(GetDlgItem(hwndDlg, IDC_BTN_DESTUP), (sel2 - 1 > 0));

+						int count = SendDlgItemMessage(hwndDlg, IDC_LST_DEST, LB_GETCOUNT, 0, 0);

+						EnableWindow(GetDlgItem(hwndDlg, IDC_BTN_DESTDOWN), (sel2 - 1 < count - 1));

+

+						SendMessage(GetParent(hwndDlg), PSM_CHANGED, 0, 0);

 					}

 				}

+			}

 

-				break;

+			break;

 			}

 		}

-		if(LOWORD(wParam) == IDC_BGCOL

+		if (LOWORD(wParam) == IDC_BGCOL

 			|| LOWORD(wParam) == IDC_SP_INDENT || LOWORD(wParam) == IDC_SP_ROWHEIGHT)

 		{

-			SendMessage( GetParent( hwndDlg ), PSM_CHANGED, 0, 0 );

+			SendMessage(GetParent(hwndDlg), PSM_CHANGED, 0, 0);

 		}

 		break;

 

 	case WM_NOTIFY:

-		if (((LPNMHDR)lParam)->code == PSN_APPLY )

+		if (((LPNMHDR)lParam)->code == PSN_APPLY)

 		{

 			CallService(PLUG "/SetAndSavePingList", (WPARAM)&temp_list, 0);

 			CallService(PLUG "/GetPingList", 0, (LPARAM)&temp_list);

@@ -497,23 +501,23 @@ static INT_PTR CALLBACK DlgProcOpts2(HWND hwndDlg, UINT msg, WPARAM wParam, LPAR
 	return FALSE;

 }

 

-int PingOptInit(WPARAM wParam,LPARAM)

+int PingOptInit(WPARAM wParam, LPARAM)

 {

 	OPTIONSDIALOGPAGE odp = { 0 };

 	odp.hInstance = hInst;

-	odp.flags = ODPF_BOLDGROUPS|ODPF_TCHAR;

+	odp.flags = ODPF_BOLDGROUPS | ODPF_TCHAR;

 	odp.ptszGroup = LPGENT("Network");

 	odp.ptszTitle = LPGENT("Ping");

 

 	odp.ptszTab = LPGENT("Settings");

 	odp.pszTemplate = MAKEINTRESOURCEA(IDD_DIALOG1);

 	odp.pfnDlgProc = DlgProcOpts;

-	Options_AddPage(wParam,&odp);

+	Options_AddPage(wParam, &odp);

 

 	odp.ptszTab = LPGENT("Hosts");

 	odp.pszTemplate = MAKEINTRESOURCEA(IDD_DIALOG2);

 	odp.pfnDlgProc = DlgProcOpts2;

-	Options_AddPage(wParam,&odp);

+	Options_AddPage(wParam, &odp);

 	return 0;

 }