alcoholic

Below are first my main file and then the port write function I have got no errors but i dont know where I am going wrong as the 3 byte protocol still wont send to the serial port!! can anyone help me achieve the impossible and send the three byte protocol??
thanks



/************************************************** *********************

WINMAIN ( HINSTANCE hInstance,
HINSTANCE hPrevInstance,
LPTSTR lpCmdLine,
int nCmdShow)

************************************************** *********************/

int WINAPI WinMain( HINSTANCE hInstance,
HINSTANCE hPrevInstance,
LPTSTR lpCmdLine,
int nCmdShow)



{


hPort = INVALID_HANDLE_VALUE; // Serial port handle
hReadThread = NULL; // Handle to the read thread

unsigned char Bytes[3] = { 255, 1, 127 } ;

lpszDevName = TEXT("COM1:");

// Initialize the port.
if (!PortInitialize (lpszDevName))
{
return 0 ;
}


PortWrite(Bytes) ;

PortClose(hPort) ;


return 0 ;

}



/************************************************** *********************

PortWrite (BYTE Byte)

************************************************** *********************/
void PortWrite (unsigned char *Bytes)
{
DWORD dwError,
dwNumBytesWritten;


if (!WriteFile (hPort, // Port handle
&Bytes, // Pointer to the data to write
3, // Number of bytes to write
&dwNumBytesWritten, // Pointer to the number of bytes
// written
NULL)) // Must be NULL for Windows CE
{
// WriteFile failed. Report error.
dwError = GetLastError ();
}
}

Dang

Its difficult to debug what you've got since it is incomplete.

I would recommend looking at http://www.programmersheaven.com/zone3/cat411/index.htm

alcoholic

could i mail you my code in full so that you can spot why it wont send the bytes to the serial port??

Dang

No problem. Just push the email button and enter your code. Please use "CODE" (replace " with [ and ] respectively) to start code and "/CODE"to end it. This will keep it much cleaner.

adrianxw

Just a simple question, are you sure you understand the passing of char * to functions?

__________________
Wave upon wave of demented avengers march cheerfully out of obscurity unto the dream.

Salem

Doesn't look like it
> void PortWrite (unsigned char *Bytes)
...
> &Bytes, // Pointer to the data to write
This isn't true
It should be Bytes or &Bytes[0]

__________________
If you dance barefoot on the broken glass of undefined behaviour, you've got to expect the occasional cut.

adrianxw

__________________
Wave upon wave of demented avengers march cheerfully out of obscurity unto the dream.