El Dimarts, 11 de juny de 2013, a les 20:34:09, Mihai Niculescu va escriure: > On 06/11/2013 08:19 PM, Albert Astals Cid wrote: > > 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 > > Since it has the issues presented above, I do not recommend to use it in > production. But I think it would be helpful for other devs who want to > improve the Arthur backend.
Are you planning to keep working on it to a stage in which in can be shipped in a future poppler release? 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 > > _______________________________________________ > poppler mailing list > [email protected] > http://lists.freedesktop.org/mailman/listinfo/poppler _______________________________________________ poppler mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/poppler
