Code review comment for lp:~schwieni/inkscape/inkscape

Revision history for this message
schwieni (schwieni) wrote :

In this case I would enclose line 1652 to 1676 from main.cpp with
"if(sp_export_pdf) { ... }" rather than only the last lines

Am 06.06.2013 10:52, schrieb Vinipsmaker:
> The first patch makes Inkscape crash when someone try to export a PS file (the affected function is used to export (E)PS and PDF files). r12352 have the right correction:
>
> http://bazaar.launchpad.net/~inkscape.dev/inkscape/trunk/revision/12352
>
> I'll try to discuss the patch more next time.

« Back to merge proposal