[Poppler-bugs] [Bug 79936] Add support for printing to a Windows printer from pdftocairo
bugzilla-daemon at freedesktop.org
bugzilla-daemon at freedesktop.org
Sat Sep 27 01:33:30 PDT 2014
https://bugs.freedesktop.org/show_bug.cgi?id=79936
--- Comment #33 from Carlos Garcia Campos <carlosgc at gnome.org> ---
Comment on attachment 106267
--> https://bugs.freedesktop.org/attachment.cgi?id=106267
Allow an output file for win32 printing to be specified
Review of attachment 106267:
-----------------------------------------------------------------
why GSView? :-P Patch LGTM
::: utils/pdftocairo-win32.cc
@@ +188,5 @@
> + docinfo.lpszDocName = inputFileName->getCString();
> + }
> + if (outputFileName) {
> + docinfo.lpszOutput = outputFileName->getCString();
> + }
Why have you added braces everywhere?
::: utils/pdftocairo.cc
@@ +741,5 @@
> }
>
> + if (print) {
> + return NULL; // No output file means print to printer
> + }
Ditto.
--
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/poppler-bugs/attachments/20140927/c918a0e3/attachment.html>
More information about the Poppler-bugs
mailing list