Re: Serial thread continued

You have to play by the rules. At least know the rules.

You HAVE to pass OVERLAPPED to WaitCommEvent. You HAVE to initialize hEvent
in OVERLAPPED.

Also, PLEASE, read in the SDK docs what FALSE return from ReadFile means,
and what special GetLastError code it could give in case of overlapped IO.

"Kahlua" <edward.frederick@verizon.net> wrote in message
news:0fad3aff-a28d-4435-a5e3-ff5e0f521253@56g2000hsm.googlegroups.com...

As usual your reply helps very little.
Your answers always point out "what not to do", but little as to "what
to do".
You allways refer to your essays.
What if I do not want to REWRITE my entire application to incorporate
all the threads you suggest.
My open serial is fine, my parameter setting is fine, and my writing
is fine.
I just want a worker thread that works, so whenever data comes in the
port I act upon it.
A different number of bytes come in depending on the 1st byte value.
I cannot simply refer to your Reader thread as it uses parts of your
Writer functions.
Also I am using VS6 and your essay doesnt.
I find it hard to believe it takes 50+ lined of code in the worker
thread just to receive a little data.
ALSO I replaced the AfxMessageBox areas with your TRACE as below.

UINT SerialThread( LPVOID Param ) //Thread to monitor serial activity
{
  HWND hDlg = (HWND)Param;
  OVERLAPPED ovl = {0};
  BYTE chread;
  DWORD dwRead;
  DWORD dwEventMask;
  int i;

  if(!SetCommMask(hCom, EV_RXCHAR))
    AfxMessageBox("Error setting ComMask");
  BOOL running = TRUE;
  while(running){
    if(WaitCommEvent(hCom, &dwEventMask, NULL)){
      if (!ReadFile (hCom, &chread, 1, &dwRead, &ovl)){
        DWORD err = ::GetLastError();
        TRACE(_T("Error reading data: %d\n"), err);
      }
      if (chread == 0x10){
        for (i=0; i<33; i++){
          if (!ReadFile (hCom, &chread, 1, &dwRead, &ovl)){
            DWORD err = ::GetLastError();
            TRACE(_T("Error reading data: %d\n"), err);
          }
          Bar[i] = chread;
        }
        ::PostMessage(hDlg, MY_SERIAL, (WPARAM)0, (LPARAM)0);
      }
    }
  }
  return TRUE;
}

Now NO errors are reported but all the data is not being received.
How did this solve anything but remove the old messages?