[poppler] Arthur Backend

[Mihai Niculescu]

On 06/11/2013 08:39 PM, Albert Astals Cid wrote:
> 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
I hope I'll have time to improve it, but I don't know for sure.
>>>> 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
> _______________________________________________
> poppler mailing list
> poppler at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/poppler