[poppler] Arthur Backend

[Albert Astals Cid]

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
> >>>>> poppler at lists.freedesktop.org
> >>>>> http://lists.freedesktop.org/mailman/listinfo/poppler
> >>>> 
> >>>> _______________________________________________
> >>>> poppler mailing list
> >>>> poppler at lists.freedesktop.org
> >>>> http://lists.freedesktop.org/mailman/listinfo/poppler
> > 
> > _______________________________________________
> > poppler mailing list
> > poppler at lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/poppler