[poppler] Arthur Backend

[Mihai Niculescu]

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

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

-------------- next part --------------
A non-text attachment was scrubbed...
Name: testPoppler-Arthur.tar.gz
Type: application/gzip
Size: 2457 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/poppler/attachments/20130611/5fcd88dc/attachment.bin>