Changeset 929 for trunk/dll/comp.c


Ignore:
Timestamp:
Jan 18, 2008, 8:19:30 PM (17 years ago)
Author:
Steven Levine
Message:

Compare directories rework.
Change hide not selected to 3 state checkbox.
Honor filters in actions.
Do all total and select count updates on WM_TIMER.

File:
1 edited

Legend:

Unmodified
Added
Removed

  • TabularUnified trunk/dll/comp.c

    r924 r929  
    4747  12 Jan 08 SHL Use SleepIfNeeded
    4848  12 Jan 08 SHL Reduce/eliminate more DosSleep calls
     49  16 Jan 08 SHL Update total/select counts with WM_TIMER only
     50  17 Jan 08 SHL Change hide not selected button to 3 state
     51  18 Jan 08 SHL Honor filters in actions
    4952
    5053***********************************************************************/
    … …  
    441444                        MPFROM2SHORT(CMA_FIRST, CMA_ITEMORDER));
    442445
    443       WinStartTimer(hab, cmp->hwnd, ID_TIMER, 2000);
    444446      InitITimer(&itdSleep, 500);               // Sleep every 500 mSec
    445447
    … …  
    451453                           MPFROM2SHORT(CMA_NEXT, CMA_ITEMORDER));
    452454
    453         if (*pciS->pszFileName && pciS->rc.flRecordAttr & CRA_SELECTED) {
    454 
     455        // Process file if selected and not filtered
     456        if (*pciS->pszFileName &&
     457             pciS->rc.flRecordAttr & CRA_SELECTED &&
     458             ~pciS->rc.flRecordAttr & CRA_FILTERED)
     459        {
    455460          // Source name not blank
    456461          switch (cmp->action) {
    … …  
    477482                           MPFROM2SHORT(1, CMA_ERASE | CMA_TEXTCHANGED));
    478483                pciD->flags = 0;        // Just on one side
    479                 if (pciD->pszSubject != NullStr) {
     484                if (pciD->pszSubject != NullStr) {
    480485                  xfree(pciD->pszSubject);
    481486                  pciD->pszSubject = NullStr;
    482                 }
     487                }
    483488              }
    484489              if (hwndCnrS == WinWindowFromID(cmp->hwnd, COMP_LEFTDIR))
    … …  
    486491              else
    487492                cmp->cmp->totalright--;
    488               // DosSleep(0);           // 8-26-07 GKY 1 // 12 Jan 08 SHL
    489493            }
    490494            break;
    … …  
    667671        SleepIfNeeded(&itdSleep, 0);
    668672      } // while
     673      WinPostMsg(cmp->hwnd, WM_TIMER, MPFROMLONG(ID_TIMER), 0); // Force update
    669674    Abort:
    670       WinStopTimer(hab, cmp->hwnd, ID_TIMER);
    671675      WinDestroyMsgQueue(hmq);
    672676    }
    673     PostMsg(cmp->hwnd, UM_CONTAINER_FILLED, MPFROMLONG(1L), MPVOID);
    674     PostMsg(cmp->hwnd, WM_COMMAND, MPFROM2SHORT(IDM_DESELECTALL, 0), MPVOID);
     677    PostMsg(cmp->hwnd, UM_CONTAINER_FILLED, MPFROMLONG(1), MPVOID);
     678    // PostMsg(cmp->hwnd, WM_COMMAND, MPFROM2SHORT(IDM_DESELECTALL, 0), MPVOID);        // 18 Jan 08 SHL we can count now
    675679    DecrThreadUsage();
    676680    WinTerminate(hab);
    … …  
    679683}
    680684
    681 VOID CompSelect(HWND hwndCnrS, HWND hwndCnrD, INT action, BOOL reset);
     685VOID CompSelect(HWND hwndCnrS, HWND hwndCnrD, HWND hwnd, INT action, BOOL reset);
    682686
    683687//=== SelectCnrsThread() Update container selection flags thread ===
    … …  
    703707      IncrThreadUsage();
    704708      priority_normal();
    705       WinStartTimer(hab, cmp->hwnd, ID_TIMER, 2000);
    706709      switch (cmp->action) {
    707710      case IDM_INVERT:
    … …  
    722725        CompSelect(WinWindowFromID(cmp->hwnd, COMP_LEFTDIR),
    723726                   WinWindowFromID(cmp->hwnd, COMP_RIGHTDIR),
    724                    cmp->action, cmp->reset);
     727                   cmp->hwnd,
     728                   cmp->action,
     729                   cmp->reset);
    725730        break;
    726731      }
    727       WinStopTimer(hab, cmp->hwnd, ID_TIMER);
    728732      if (!PostMsg(cmp->hwnd, UM_CONTAINER_FILLED, MPFROMLONG(1L), MPVOID))
    729733        WinSendMsg(cmp->hwnd, UM_CONTAINER_FILLED, MPFROMLONG(1L), MPVOID);
    … …  
    742746 */
    743747
    744 VOID CompSelect(HWND hwndCnrS, HWND hwndCnrD, INT action, BOOL reset)
     748VOID CompSelect(HWND hwndCnrS, HWND hwndCnrD, HWND hwnd, INT action, BOOL reset)
    745749{
    746750  PCNRITEM pciS, pciD, *pciSa = NULL, *pciDa = NULL;
    … …  
    750754  INT ret = 0;
    751755  ITIMER_DESC itdSleep = { 0 };
     756  BOOL fUpdateHideButton = FALSE;
    752757
    753758  if (!hwndCnrS || !hwndCnrD) {
    … …  
    11411146      if (~pciSa[x]->rc.flRecordAttr & CRA_FILTERED &&
    11421147          pciSa[x]->flags & CNRITEM_EXISTS) {
    1143         if (pciSa[x]->rc.flRecordAttr & CRA_SELECTED)
     1148        if (pciSa[x]->rc.flRecordAttr & CRA_SELECTED) {
    11441149          WinSendMsg(hwndCnrS, CM_SETRECORDEMPHASIS, MPFROMP(pciSa[x]),
    11451150                     MPFROM2SHORT(FALSE, CRA_SELECTED));
    1146         if (pciDa[x]->rc.flRecordAttr & CRA_SELECTED)
     1151          fUpdateHideButton = TRUE;
     1152        }
     1153        if (pciDa[x]->rc.flRecordAttr & CRA_SELECTED) {
    11471154          WinSendMsg(hwndCnrD, CM_SETRECORDEMPHASIS, MPFROMP(pciDa[x]),
    11481155                     MPFROM2SHORT(FALSE, CRA_SELECTED));
     1156          fUpdateHideButton = TRUE;
     1157        }
    11491158      }
    11501159      SleepIfNeeded(&itdSleep, 0);
    … …  
    11561165      if (~pciSa[x]->rc.flRecordAttr & CRA_FILTERED) {
    11571166        if (~pciSa[x]->flags & CNRITEM_EXISTS) {
    1158         if (*pciSa[x]->pszFileName) {
    1159           if (pciSa[x]->rc.flRecordAttr & CRA_SELECTED)
    1160             WinSendMsg(hwndCnrS, CM_SETRECORDEMPHASIS, MPFROMP(pciSa[x]),
    1161                        MPFROM2SHORT(FALSE, CRA_SELECTED));
    1162         }
    1163         else if (pciDa[x]->rc.flRecordAttr & CRA_SELECTED)
     1167          if (*pciSa[x]->pszFileName) {
     1168            if (pciSa[x]->rc.flRecordAttr & CRA_SELECTED) {
     1169              WinSendMsg(hwndCnrS, CM_SETRECORDEMPHASIS, MPFROMP(pciSa[x]),
     1170                         MPFROM2SHORT(FALSE, CRA_SELECTED));
     1171              fUpdateHideButton = TRUE;
     1172            }
     1173          }
     1174          else if (pciDa[x]->rc.flRecordAttr & CRA_SELECTED) {
    11641175            WinSendMsg(hwndCnrD, CM_SETRECORDEMPHASIS, MPFROMP(pciDa[x]),
    11651176                       MPFROM2SHORT(FALSE, CRA_SELECTED));
     1177            fUpdateHideButton = TRUE;
     1178          }
    11661179        }
    11671180      }
    … …  
    11741187      if (~pciSa[x]->rc.flRecordAttr & CRA_FILTERED) {
    11751188        if (pciSa[x]->flags & CNRITEM_LARGER) {
    1176           if (pciSa[x]->rc.flRecordAttr & CRA_SELECTED)
     1189          if (pciSa[x]->rc.flRecordAttr & CRA_SELECTED) {
    11771190            WinSendMsg(hwndCnrS, CM_SETRECORDEMPHASIS, MPFROMP(pciSa[x]),
    11781191                       MPFROM2SHORT(FALSE, CRA_SELECTED));
     1192            fUpdateHideButton = TRUE;
     1193          }
    11791194        }
    11801195        else if (pciDa[x]->flags & CNRITEM_LARGER) {
    1181           if (pciDa[x]->rc.flRecordAttr & CRA_SELECTED)
     1196          if (pciDa[x]->rc.flRecordAttr & CRA_SELECTED) {
    11821197            WinSendMsg(hwndCnrD, CM_SETRECORDEMPHASIS, MPFROMP(pciDa[x]),
    11831198                       MPFROM2SHORT(FALSE, CRA_SELECTED));
     1199            fUpdateHideButton = TRUE;
     1200          }
    11841201        }
    11851202      }
    … …  
    11921209      if (~pciSa[x]->rc.flRecordAttr & CRA_FILTERED) {
    11931210        if (pciSa[x]->flags & CNRITEM_SMALLER) {
    1194           if (pciSa[x]->rc.flRecordAttr & CRA_SELECTED)
     1211          if (pciSa[x]->rc.flRecordAttr & CRA_SELECTED) {
    11951212            WinSendMsg(hwndCnrS, CM_SETRECORDEMPHASIS, MPFROMP(pciSa[x]),
    11961213                       MPFROM2SHORT(FALSE, CRA_SELECTED));
     1214            fUpdateHideButton = TRUE;
     1215          }
    11971216        }
    11981217        else if (pciDa[x]->flags & CNRITEM_SMALLER) {
    1199           if (pciDa[x]->rc.flRecordAttr & CRA_SELECTED)
     1218          if (pciDa[x]->rc.flRecordAttr & CRA_SELECTED) {
    12001219            WinSendMsg(hwndCnrD, CM_SETRECORDEMPHASIS, MPFROMP(pciDa[x]),
    12011220                       MPFROM2SHORT(FALSE, CRA_SELECTED));
     1221            fUpdateHideButton = TRUE;
     1222          }
    12021223        }
    12031224      }
    … …  
    12101231      if (~pciSa[x]->rc.flRecordAttr & CRA_FILTERED) {
    12111232        if (pciSa[x]->flags & CNRITEM_NEWER) {
    1212           if (pciSa[x]->rc.flRecordAttr & CRA_SELECTED)
     1233          if (pciSa[x]->rc.flRecordAttr & CRA_SELECTED) {
    12131234            WinSendMsg(hwndCnrS, CM_SETRECORDEMPHASIS, MPFROMP(pciSa[x]),
    12141235                       MPFROM2SHORT(FALSE, CRA_SELECTED));
     1236            fUpdateHideButton = TRUE;
     1237          }
    12151238        }
    12161239        else if (pciDa[x]->flags & CNRITEM_NEWER) {
    1217           if (pciDa[x]->rc.flRecordAttr & CRA_SELECTED)
     1240          if (pciDa[x]->rc.flRecordAttr & CRA_SELECTED) {
    12181241            WinSendMsg(hwndCnrD, CM_SETRECORDEMPHASIS, MPFROMP(pciDa[x]),
    12191242                       MPFROM2SHORT(FALSE, CRA_SELECTED));
     1243            fUpdateHideButton = TRUE;
     1244          }
    12201245        }
    12211246      }
    … …  
    12281253      if (~pciSa[x]->rc.flRecordAttr & CRA_FILTERED) {
    12291254        if (pciSa[x]->flags & CNRITEM_OLDER) {
    1230           if (pciSa[x]->rc.flRecordAttr & CRA_SELECTED)
     1255          if (pciSa[x]->rc.flRecordAttr & CRA_SELECTED) {
    12311256            WinSendMsg(hwndCnrS, CM_SETRECORDEMPHASIS, MPFROMP(pciSa[x]),
    12321257                       MPFROM2SHORT(FALSE, CRA_SELECTED));
     1258            fUpdateHideButton = TRUE;
     1259          }
    12331260        }
    12341261        else if (pciDa[x]->flags & CNRITEM_OLDER) {
    1235           if (pciDa[x]->rc.flRecordAttr & CRA_SELECTED)
     1262          if (pciDa[x]->rc.flRecordAttr & CRA_SELECTED) {
    12361263            WinSendMsg(hwndCnrD, CM_SETRECORDEMPHASIS, MPFROMP(pciDa[x]),
    12371264                       MPFROM2SHORT(FALSE, CRA_SELECTED));
     1265            fUpdateHideButton = TRUE;
     1266          }
    12381267        }
    12391268      }
    … …  
    12441273  default:
    12451274    break;
    1246   }
     1275  } // switch
    12471276
    12481277  if (reset) {
    … …  
    12501279      WinSendMsg(hwndCnrS, CM_INVALIDATERECORD,
    12511280                 MPFROMP(pciSa), MPFROM2SHORT((min(numS, 65535)), 0));
    1252       // DosSleep(0);                   //Let screen update // 12 Jan 08 SHL
    12531281      WinSendMsg(hwndCnrD, CM_INVALIDATERECORD,
    12541282                 MPFROMP(pciDa), MPFROM2SHORT((min(numD, 65535)), 0));
    12551283      numS -= min(numS, 65535);
    1256       // if (numS) // 12 Jan 08 SHL
    1257         // DosSleep(0); //26 Aug 07 GKY 1
    12581284      SleepIfNeeded(&itdSleep, 0); // 12 Jan 08 SHL
    12591285    } // while
    … …  
    12621288  free(pciSa);
    12631289  free(pciDa);
     1290
     1291  if (fUpdateHideButton) {
     1292    if (WinQueryButtonCheckstate(hwnd,COMP_HIDENOTSELECTED) == 1)
     1293      WinCheckButton(hwnd, COMP_HIDENOTSELECTED, 2);
     1294  }
     1295
     1296  WinPostMsg(hwnd, WM_TIMER, MPFROMLONG(ID_TIMER), 0);  // Force update
    12641297  DosPostEventSem(CompactSem);
    12651298}
    … …  
    14381471      WinCancelShutdown(hmq, TRUE);
    14391472      IncrThreadUsage();
    1440       WinStartTimer(hab, cmp->hwnd, ID_TIMER, 2000);
    14411473
    14421474      hwndLeft = WinWindowFromID(cmp->hwnd, COMP_LEFTDIR);
    … …  
    19922024      } // if recsNeeded
    19932025
    1994       WinStopTimer(hab, cmp->hwnd, ID_TIMER);
    1995 
    19962026      Deselect(hwndLeft);
    19972027      Deselect(hwndRight);
    … …  
    20242054}
    20252055
    2026 // fixme to be gone - use variable
     2056// fixme to be gone - use variable?
    20272057#define hwndLeft        (WinWindowFromID(hwnd,COMP_LEFTDIR))
    20282058#define hwndRight       (WinWindowFromID(hwnd,COMP_RIGHTDIR))
    … …  
    20682098        }
    20692099      }
     2100      WinStartTimer(WinQueryAnchorBlock(hwnd), hwnd, ID_TIMER, 500);
    20702101    }
    20712102    break;
    … …  
    22182249        pci = (PCNRITEM)pcown->pRecord;
    22192250        // 01 Aug 07 SHL if field null or blank, we draw
    2220         // fixme to know why - probably to optimize and bypass draw?
     2251        // fixme to document why - probably to optimize and bypass draw?
    22212252        if (pci && (INT)pci != -1 && !*pci->pszFileName)
    22222253          return MRFROMLONG(TRUE);
    … …  
    22432274    }
    22442275    else {
    2245       // 05 Jan 08 SHL fixme to use timer id to optimize output
    2246       sprintf(s, " %d", cmp->totalleft);
    2247       WinSetDlgItemText(hwnd, COMP_TOTALLEFT, s);
    2248       sprintf(s, " %d", cmp->totalright);
    2249       WinSetDlgItemText(hwnd, COMP_TOTALRIGHT, s);
    2250       sprintf(s, " %d", cmp->selleft);
    2251       WinSetDlgItemText(hwnd, COMP_SELLEFT, s);
    2252       sprintf(s, " %d", cmp->selright);
    2253       WinSetDlgItemText(hwnd, COMP_SELRIGHT, s);
     2276      if (cmp->uOldTotalLeft != cmp->totalleft) {
     2277        cmp->uOldTotalLeft = cmp->totalleft;
     2278        sprintf(s, " %d", cmp->totalleft);
     2279        WinSetDlgItemText(hwnd, COMP_TOTALLEFT, s);
     2280      }
     2281      if (cmp->uOldTotalRight != cmp->totalright) {
     2282        cmp->uOldTotalRight = cmp->totalright;
     2283        sprintf(s, " %d", cmp->totalright);
     2284        WinSetDlgItemText(hwnd, COMP_TOTALRIGHT, s);
     2285      }
     2286      if (cmp->uOldSelLeft != cmp->selleft) {
     2287        cmp->uOldSelLeft = cmp->selleft;
     2288        sprintf(s, " %d", cmp->selleft);
     2289        WinSetDlgItemText(hwnd, COMP_SELLEFT, s);
     2290      }
     2291      if (cmp->uOldSelRight != cmp->selright) {
     2292        cmp->uOldSelRight = cmp->selright;
     2293        sprintf(s, " %d", cmp->selright);
     2294        WinSetDlgItemText(hwnd, COMP_SELRIGHT, s);
     2295      }
    22542296    }
    22552297    break;
    … …  
    22672309      WinEnableWindowUpdate(hwndLeft, TRUE);
    22682310      WinEnableWindowUpdate(hwndRight, TRUE);
    2269       sprintf(s, " %d", cmp->totalleft);
    2270       WinSetDlgItemText(hwnd, COMP_TOTALLEFT, s);
    2271       sprintf(s, " %d", cmp->totalright);
    2272       WinSetDlgItemText(hwnd, COMP_TOTALRIGHT, s);
    2273       sprintf(s, " %d", cmp->selleft);
    2274       WinSetDlgItemText(hwnd, COMP_SELLEFT, s);
    2275       sprintf(s, " %d", cmp->selright);
    2276       WinSetDlgItemText(hwnd, COMP_SELRIGHT, s);
    2277       // 12 Jan 08 SHL fixme to have SetEnables(COMPARE* pcmp, BOOL fEnable)
     2311      WinPostMsg(hwnd, WM_TIMER, MPFROMLONG(ID_TIMER), 0);      // Force update
     2312      // 12 Jan 08 SHL fixme to have SetButtonEnables(COMPARE* pcmp, BOOL fEnable)
    22782313      // to replace duplicated code here and elsewhere
    22792314      WinEnableWindow(WinWindowFromID(hwnd, DID_OK), TRUE);
    … …  
    24522487      case CN_BEGINEDIT:
    24532488      case CN_REALLOCPSZ:
    2454         // fixme to be gone - field edits not allowed
     2489        // fixme to be gone - field edits not allowed?
    24552490        Runtime_Error(pszSrcFile, __LINE__,
    24562491                      "CN_BEGINEDIT/CN_REALLOCPSZ unexpected");
    … …  
    24602495        {
    24612496          PNOTIFYRECORDEMPHASIS pnre = mp2;
     2497          BOOL fSelected;
    24622498          if (pnre->fEmphasisMask & CRA_SELECTED) {
     2499            // Select toggled
    24632500            PCNRITEM pci = (PCNRITEM)pnre->pRecord;
    24642501            if (pci) {
    24652502              if (!*pci->pszFileName) {
     2503                // Slot empty
     2504                // 17 Jan 08 SHL fixme to know how can get here
    24662505                // 12 Jan 08 SHL fixme to know if select counts need update?
    24672506                if (pci->rc.flRecordAttr & CRA_SELECTED)
    … …  
    24722511              }
    24732512              else {
     2513                BOOL fUpdateHideButton = FALSE;
    24742514                cmp = INSTDATA(hwnd);
    24752515                if (SHORT1FROMMP(mp1) == COMP_LEFTDIR) {
    2476                   cmp->selleft +=
    2477                     pci->rc.flRecordAttr & CRA_SELECTED ? 1 : -1;
    2478                   // If window not enabled WM_TIMER will update display
    2479                   if (WinIsWindowEnabled(hwndLeft)) {
    2480                     sprintf(s, " %d", cmp->selleft);
    2481                     WinSetDlgItemText(hwnd, COMP_SELLEFT, s);
    2482                   }
     2516                  fSelected = pci->rc.flRecordAttr & CRA_SELECTED;
     2517                  cmp->selleft += fSelected ? 1 : -1;
     2518                  if (!fSelected)
     2519                    fUpdateHideButton = TRUE;
    24832520                }
    24842521                else if (SHORT1FROMMP(mp1) == COMP_RIGHTDIR) {
    2485                   cmp->selright +=
    2486                     pci->rc.flRecordAttr & CRA_SELECTED ? 1 : -1;
    2487                   if (WinIsWindowEnabled(hwndRight)) {
    2488                     sprintf(s, " %d", cmp->selright);
    2489                     WinSetDlgItemText(hwnd, COMP_SELRIGHT, s);
    2490                   }
     2522                  fSelected = pci->rc.flRecordAttr & CRA_SELECTED;
     2523                  cmp->selright += fSelected ? 1 : -1;
     2524                  if (!fSelected)
     2525                    fUpdateHideButton = TRUE;
    24912526                }
    24922527                else {
    24932528                  Runtime_Error(pszSrcFile, __LINE__,
    24942529                                "mp1 %u unexpected", SHORT1FROMMP(mp1));
     2530                }
     2531                if (fUpdateHideButton) {
     2532                  USHORT state = WinQueryButtonCheckstate(hwnd,COMP_HIDENOTSELECTED);
     2533                  if (state == 1) {
     2534                    WinCheckButton(hwnd, COMP_HIDENOTSELECTED, 2);
     2535                  }
    24952536                }
    24962537              }
    … …  
    25612602          WinEnableWindowUpdate(hwndRight, FALSE);
    25622603          cmp->selleft = cmp->selright = 0;
    2563           WinSetDlgItemText(hwnd, COMP_SELLEFT, "0");
    2564           WinSetDlgItemText(hwnd, COMP_SELRIGHT, "0");
    2565           WinSetDlgItemText(hwnd, COMP_TOTALLEFT, "0");
    2566           WinSetDlgItemText(hwnd, COMP_TOTALRIGHT, "0");
    25672604          WinSetDlgItemText(hwnd, COMP_NOTE,
    25682605                            GetPString(IDS_COMPHOLDREADDISKTEXT));
    … …  
    26362673    cmp = INSTDATA(hwnd);
    26372674    if (cmp) {
    2638       USHORT wantHide = WinQueryButtonCheckstate(hwnd,
    2639                                                  COMP_HIDENOTSELECTED);
     2675      USHORT wasHidden = WinQueryButtonCheckstate(hwnd,
     2676                                                  COMP_HIDENOTSELECTED);
    26402677
    26412678      // cmp->dcd.suspendview = 1;              // 12 Jan 08 SHL appears not to be used here
    2642       if (wantHide) {
     2679      if (wasHidden != 1) {
     2680        // Hide if not selected on both sides
    26432681        BOOL needRefresh = FALSE;
    26442682        HWND hwndl = WinWindowFromID(cmp->hwnd, COMP_LEFTDIR);
    … …  
    26522690          if (~pcil->rc.flRecordAttr & CRA_SELECTED &&
    26532691              ~pcir->rc.flRecordAttr & CRA_SELECTED) {
     2692            // 17 Jan 08 SHL fixme to optimize refresh
    26542693            pcil->rc.flRecordAttr |= CRA_FILTERED;
    26552694            pcir->rc.flRecordAttr |= CRA_FILTERED;
    … …  
    26692708      }
    26702709      else {
     2710        // Unhide
    26712711        WinSendMsg(hwndLeft, CM_FILTER, MPFROMP(Filter),
    26722712                   MPFROMP(&cmp->dcd.mask));
    … …  
    26832723      else
    26842724        WinSetDlgItemText(hwnd, COMP_NOTE, GetPString(IDS_COMPREADYTEXT));
     2725      WinCheckButton(hwnd, COMP_HIDENOTSELECTED, wasHidden != 1 ? 1 : 0);
    26852726    }
    26862727    return 0;
    … …  
    30913132              TileChildren(cmp->hwndParent, TRUE);
    30923133            }
    3093             // DosSleep(32);            // 05 Aug 07 GKY 64
    30943134            DosSleep(1);                // 12 Jan 08 SHL Let screen update
    30953135            PostMsg(hwnd, WM_COMMAND, MPFROM2SHORT(COMP_COLLECT, 0), MPVOID);
    … …  
    31353175
    31363176  case WM_CLOSE:
     3177    // 18 Jan 08 SHL fixme to hold off if thread busy?
    31373178    WinDismissDlg(hwnd, 0);
    31383179    return 0;
    … …  
    31413182    cmp = INSTDATA(hwnd);
    31423183    if (cmp) {
     3184      // 17 Jan 08 SHL fixme to know if stop really needed?
     3185      WinStopTimer(WinQueryAnchorBlock(hwnd), hwnd, ID_TIMER);
    31433186      if (cmp->dcd.hwndLastMenu)
    31443187        WinDestroyWindow(cmp->dcd.hwndLastMenu);
Note: See TracChangeset for help on using the changeset viewer.