Hello Mark,
I took a look at your program and have some question / advice for you.
First of all, I wonder why you choose to go the pointer way in using TSQs instead of simpler CmtWriteTSQData / CmtReadTSQData mechanism, which takes care by itself of flushing the queue. Using pointers you can have interlocking problems like in your AcquireTSQDataWithPtrsThreadFunction function:
static int CVICALLBACK AcquireTSQDataWithPtrsThreadFunction (void *unused)
{
int numToWrite;
int index;
char *writePtr;
while (running)
{
/* Use direct pointers to write data to the thread-safe queue. */
if (CmtGetTSQWritePtr (tsqHandle, &writePtr, &numToWrite) >= 0)
{
/* Ensure we do not overflow the queue. */
if (numToWrite > RAW_DATA_Q_SIZE)
numToWrite = RAW_DATA_Q_SIZE;
//FlushInQ(comport);
//ProcessSystemEvents();
ComRd (comport, //comport to be read
writePtr, //Result Buffer
RAW_DATA_Q_SIZE-1); //Number of bytes read
CmtReleaseTSQWritePtr (tsqHandle, numToWrite);
}
}
return 0;
}
In case numToWrite is equal to RAW_DATA_Q_SIZE, infact (the most possible case since the queue is created with that exact size and never grows), you are getting the pointer but you are not releasing it!
Secondly, you are setting no timeout in reading from the COM port, so you can have some (long) wait in case external device does not transmit data fast enough (default timeout time in RS232 library is 5 seconds). Use SetComTime to set a proper timeout for reading and add some error checking to ComRd to properly handle the timeout event.
Finally, while manipulating UIR objects (Set/GetCtrlVal and so on) the first parameter passed to the functions must be a panel handle (returned by LoadPanel in the main in your case). You get no errors now since you are basically in single-panel application, but this is only a case and you will get errors when adding panel to it. Either use 'panelHandle' static variable or use 'panel' callback variable, which holds the handle of the panel the control is on: in your case those variables reference to the same handle.
