El Dimarts, 11 de juny de 2013, a les 08:19:34, Mihai Niculescu va escriure: > On 06/11/2013 12:56 AM, Albert Astals Cid wrote: > > El Dilluns, 10 de juny de 2013, a les 23:39:26, Mihai Niculescu va escriure: > >> On 06/10/2013 09:13 PM, Mihai Niculescu wrote: > >>> On 06/10/2013 08:32 PM, Albert Astals Cid wrote: > >>>> El Dilluns, 3 de juny de 2013, a les 09:41:41, Mihai Niculescu va > >>>> > >>>> escriure: > >>>>> Hi, > >>>>> > >>>>> I am working on Arthur backend and did some changes for the backend > >>>>> not > >>>>> to use Splash, but only Qt and QPainter. > >>>>> Right now, the backend can render very good PDF files with the > >>>>> exceptions: > >>>>> - does not render non-latin characters > >>>>> - does not render multiple resolutions > >>>> > >>>> BTW, are you aware we are releasing a new verison really soon and > >>>> that if you > >>>> want your changes in you should send your patches *now*? > >>>> > >>>> Cheers, > >>>> > >>>> Albert > >>> > >>> Hi, > >>> > >>> I didnt use git, so I must get the trunk from git, apply the > >>> modifications and create the patch and later I'll send you the patch. > >>> I hope it is not too late. > >>> > >>> Cheers, > >>> Mihai > >> > >> I attached the patch file. This patch removes any code in Arthur's dev > >> that is dependent on Splash. It also sets a transparent background for > >> the page - this feature is what I need a lot. A user can set manually > >> the background color by himself in a QWidget/QImage. Maybe a better > >> approach is to set it to the color of the document. I'll look into this > >> in the future. > >> > >> As mentioned above, my tests showed that it can not display non-latin > >> chars and also it displays only at a single resolution. > > > > I'll try have a look before the feature freeze (in two weeks) but someone > > else other than Mihai doing some testing/code review would be awesome. > > > > Cheers, > > > > Albert > > I think I wasn't clear enough about the patch with one thing, so let me > give a better explanation. It can render a PDF page correctly only when > not setting any resolution, like this : > page->renderToPainter(&painter); > > So, testing the patch with demos/poppler_qt4viewer will not render > correctly. Instead, you should test it with the example I have attached > now. To compile it, you need to set the POPPLER variable from > testPoppler-Arthur.pro
Oh, so it's more a work in progress thing than a finished thing, right? If so we can wait for the next release then. Cheers, Albert > > Cheers, > Mihai > > >> Cheers, > >> Mihai > >> > >>>>> These are the only problems I found. Are you interested in these > >>>>> changes > >>>>> and how can I add these to git repo? > >>>>> Also, I am thinking of modifying the wrapper to also support Qt5 but I > >>>>> dont know how to modify the build system (configure, makefiles) to > >>>>> search for Qt5/Qt4, set libraries and moc accordingly and to set a > >>>>> global variable to use it in the code in order to know which code to > >>>>> enable: > >>>>> #ifdef QT5_VERSION > >>>>> > >>>>> or something like this. If someone can help me with this, than I'll > >>>>> make > >>>>> poppler-qt4 support even Qt5. > >>>>> > >>>>> Cheers, > >>>>> Mihai > >>>>> _______________________________________________ > >>>>> poppler mailing list > >>>>> [email protected] > >>>>> http://lists.freedesktop.org/mailman/listinfo/poppler > >>>> > >>>> _______________________________________________ > >>>> poppler mailing list > >>>> [email protected] > >>>> http://lists.freedesktop.org/mailman/listinfo/poppler > > > > _______________________________________________ > > poppler mailing list > > [email protected] > > http://lists.freedesktop.org/mailman/listinfo/poppler _______________________________________________ poppler mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/poppler
