diff options
| author | George Hazan <george.hazan@gmail.com> | 2013-04-09 20:03:46 +0000 | 
|---|---|---|
| committer | George Hazan <george.hazan@gmail.com> | 2013-04-09 20:03:46 +0000 | 
| commit | bcb27264ba737778e5d3edad36088bacf74f0236 (patch) | |
| tree | fd1f57744dd380b7babe312a0ab5dc60b48854f2 /plugins/AVS/src | |
| parent | 940231dc5a484b03a278900e1880aa083472b601 (diff) | |
- short function names allows to write database loops in one string;
- 'continue' operator can be used then;
- multiple bugs fixed in clists;
- code becomes much more compact;
git-svn-id: http://svn.miranda-ng.org/main/trunk@4403 1316c22d-e87f-b044-9b9b-93d7a3e3ba9c
Diffstat (limited to 'plugins/AVS/src')
| -rw-r--r-- | plugins/AVS/src/options.cpp | 35 | 
1 files changed, 14 insertions, 21 deletions
| diff --git a/plugins/AVS/src/options.cpp b/plugins/AVS/src/options.cpp index 46908870fa..4cd2748660 100644 --- a/plugins/AVS/src/options.cpp +++ b/plugins/AVS/src/options.cpp @@ -423,31 +423,24 @@ INT_PTR CALLBACK DlgProcOptionsProtos(HWND hwndDlg, UINT msg, WPARAM wParam, LPA  		case 0:
  			switch (((LPNMHDR) lParam)->code) {
  			case PSN_APPLY:
 -				{
 -					for (int i = 0; i < ListView_GetItemCount(hwndList); i++) {
 -						char *szProto = GetProtoFromList(hwndDlg, i);
 +				for (int i = 0; i < ListView_GetItemCount(hwndList); i++) {
 +					char *szProto = GetProtoFromList(hwndDlg, i);
 -						BOOL oldVal = db_get_b(NULL, AVS_MODULE, szProto, 1);
 -						BOOL newVal = ListView_GetCheckState(hwndList, i);
 +					BOOL oldVal = db_get_b(NULL, AVS_MODULE, szProto, 1);
 +					BOOL newVal = ListView_GetCheckState(hwndList, i);
 -						if (oldVal && !newVal)
 -						{
 -							HANDLE hContact = db_find_first();
 -							while (hContact != NULL)
 -							{
 -								char* szContactProto = GetContactProto(hContact);
 -								if (szContactProto != NULL && !strcmp(szContactProto, szProto))
 -									DeleteAvatarFromCache(hContact, TRUE);
 -
 -								hContact = db_find_next(hContact);
 -							}
 +					if (oldVal && !newVal) {
 +						for (HANDLE hContact = db_find_first(); hContact; hContact = db_find_next(hContact)) {
 +							char *szContactProto = GetContactProto(hContact);
 +							if (szContactProto != NULL && !strcmp(szContactProto, szProto))
 +								DeleteAvatarFromCache(hContact, TRUE);
  						}
 -
 -						if (newVal)
 -							db_set_b(NULL, AVS_MODULE, szProto, 1);
 -						else
 -							db_set_b(NULL, AVS_MODULE, szProto, 0);
  					}
 +
 +					if (newVal)
 +						db_set_b(NULL, AVS_MODULE, szProto, 1);
 +					else
 +						db_set_b(NULL, AVS_MODULE, szProto, 0);
  				}
  			}
  		}
 | 
