[poppler] Arthur Backend

[Albert Astals Cid]

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
> >>>>>>> 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
> > 
> > _______________________________________________
> > 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