🚀 go-pugleaf

"John Scott" <jscott@kw.igs.net> wrote:
> [...] Could you point out the error?

A couple things. The pointer parameters must point to an
allocated buffer. You can dyamically allocate one chunk from
the heap and assign the pointers:

    char* ADevice = new char[ 256 * 3 ];
    char* ADriver = (char*)ADevice + 256;
    char* APort   = (char*)ADriver + 256;

The casting most likely isn't needed but I can't help myself.
Don't forget to delete [] ADevice;

OR ... you can allocate it from the stack:

    char ADevice[ 256 ] = { 0 };
    char ADriver[ 256 ] = { 0 };
    char APort[ 256 ]   = { 0 };

>    int  ADeviceMode;

Although an int is most likely ok here, it should be a THandle:

    THandle ADeviceMode;

>    DEVMODE DMode;

Get rid of this line.

>    Printer()->GetPrinter(ADevice, ADriver, APort, DMode);

Replace the DMode parameter with ADeviceMode:

    Printer()->GetPrinter(ADevice, ADriver, APort, ADeviceMode);

>    Printer()->SetPrinter(ADevice, ADriver, APort, DMode);

Same here:

    Printer()->SetPrinter(ADevice, ADriver, APort, ADeviceMode);

You also didn't show that you were setting the Printer()->PrinterIndex.
The final block of code should look something like this:

    char ADevice[ 256 ] = { 0 };
    char ADriver[ 256 ] = { 0 };
    char APort[ 256 ]   = { 0 };
    THandle ADeviceMode;

    Printer()->PrinterIndex = a valid printer number
    Printer()->GetPrinter( ADevice, ADriver, APort, ADeviceMode );
    Printer()->SetPrinter( ADevice, ADriver, APort, ADeviceMode );

~ JD