🚀 go-pugleaf

Yilmaz wrote:

> void __fastcall TForm1::Button1Click(TObject *Sender)
> {
> // Opening the Communication resource for overlapped operation
> 
> 
>   SerialPort.hComm = NULL;
>   SerialPort.hComm = CreateFile(ComboBox1->Text.c_str(),  // Pointer to the
> name of the port
>                                 GENERIC_WRITE | GENERIC_READ, // Access
> (read-write) mode
>                                 0,             // Share mode
>                                 NULL,          // Pointer to the security
> attribute
>                                 OPEN_EXISTING, // How to open the serial
> port
>                                 0,             // Port attributes
>                                 0);            // Handle to port with
> attribute to copy

Last parameter is NULL so you're *NOT* opening your Com port for 
OVERLAPPED I/O. But it's not related to your problem.



>     //Read data to the Serial Port
>     inread=ReadFile(SerialPort.hComm,outbuffer,1,&dwBytesRead,NULL);
>     if(!inread){
>     Application->MessageBox("Unable to Read Serial Port","eRRor",MB_OK);
>     return; }
> 
>     Memo->Lines->Add("Data sent to Serial");

Where do you define outbuffer? And why read (or write) one byte at a 
time. It's more efficient to read a chunk of bytes.

char outbuffer[256];
inread=ReadFile(SerialPort.hComm,outbuffer,sizeof(outbuffer),&dwBytesRead,NULL);
if(!inread)//Error
if(dwBytesRead<sizeof(outbuffer))
         outbuffer[dwBytesRead]='\0';

Michel
-- 
----------------------------------------
Michel Leunen
mailto:michel@leunen.com
http://www.leunen.com/cbuilder/
----------------------------------------