Try to Implement my first Stub function - AbortPrinter() - (try 2).

[Dmitry Timoshkov]

Loïc Maury <lmaury at gmail.com> wrote:

> After the various comments, I have modified the patch.

First of all set your tab size to 8, and ask your editor to not use tabs
at all.

> +	TRACE("(%s, %s, %p, %d, %d)\n",debugstr_w(printer->name)
> +		  ,debugstr_w(printer->printername)
> +		  ,printer->backend_printer
> +		  ,printer->queue->ref
> +		  ,list_count(&printer->queue->jobs));

TRACE() with the API parameters usually is the very first statement in
the API implementation, comma should be placed at the end of the statement,
not before.

> +				GetPrinterW(hPrinter, 2, NULL, 0, &needed);
> +				pi2 = HeapAlloc(GetProcessHeap(), 0, needed);
> +				GetPrinterW(hPrinter, 2, (LPBYTE)pi2, needed, &needed);

You need to check the return value of GetPrinterW() and handle the errors.

> +	if(pi2)
> +		HeapFree(GetProcessHeap(), 0, pi2);

NULL check before HeapFree() is not needed.

> + TRACE("return %d\n", ret);

This trace is redundant.

-- 
Dmitry.