[poppler] Creating a Beta 3

[Thomas Freitag]

Am 23.11.2012 06:39, schrieb Adam Reichold:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
> Hello,
>
> Not sure if this question is intended for everybody on the list or
> only the developers, but I'll just assume it's for everyone:
>
> Looking at the feature list and reading the bug tracker, I'd say that
> thread-safety needs to be rethought. Maybe one could make this a
> priority for the next release cycle, trying to implement the long-term
> solution of removing PDFDoc::getXRef and better distributing the
> workload, maybe under Thomas' and/or Albert's supervision. (I would be
> happy to help as much as I can, but I am not very knowledgeable about
> Poppler's internals yet.)
I had yesterday a long, long day working on getting annotations thread 
safe, and I recognized several things:
1. We can't remove PDFDoc::getXRef completely: we have the Gfx:getXRef 
only during rendering, but there are several different tasks done with 
poppler, i.e. getting information, manipulation annotations, rewrite 
PDF, etc. etc. This all must be done over PDFDoc:getXRef
2. I tried to copy the annotations for every thread, but after 
succeeding rendering became MUCH too slow.
3. Therefore I shared annotations over the threads (via the catalog), 
this was a also bigger task but at least successfully.
Atfer completing that I made several tests:
1. Rendering with and without the patch takes the same time
2. Rendering with the patch but only one thread is nearly as fast as 
without the patch (It always took the same number of seconds for multi 
page PDF)
3. Rendering with more than one thread decrease the time for rendering, 
the more the complex the PDF is.

So the latest patch on 
https://bugs.freedesktop.org/show_bug.cgi?id=50992 does what is should 
do (I missed the notification e-mail for my latest comments, so I'm nor 
sure if everbody else got them), so if Albert is able to test it before 
beta 3 it would be nice to have it. Otherwise with my changes of 
yesterday the patch again became quite huge, so I would understand if 
Albert want to suspend it 'til next year.

Cheers,
Thomas
>
> Still, I would argue for a third beta since the RTL stuff seems almost
> like a bug and less like a missing feature and probably affects a lot
> of people. (I would also include #2981 here, which depends on #55977.)
>
> Best regards, Adam.
>
> Am 23.11.2012 00:51, schrieb Albert Astals Cid:
>> Hi guys, if you look at
>> http://freedesktop.org/wiki/Software/poppler you'll see that there
>> are 5 features listed for 0.22 and that I just released 0.21.2 that
>> was supposed to be the last release with new features.
>>
>> I don't think we'll be getting all 5 in, but if we create a new
>> Beta 3 with let's say 1 week more of time maybe we can get 1 or 2
>> more in.
>>
>> What do you think?
>>
>> Cheers, Albert _______________________________________________
>> poppler mailing list poppler at lists.freedesktop.org
>> http://lists.freedesktop.org/mailman/listinfo/poppler
>>
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v2.0.19 (GNU/Linux)
>
> iQEcBAEBAgAGBQJQrwwUAAoJEPSSjE3STU34/FIIAJw3BBtbn/S4Rb1o8JOw8zF6
> 6TgNOdBKZzoa7WaYm12slVIZf3EiyaDn2NcjPb9EWaW42pvzMO2h1BDWwkUnagOD
> 190SMMyp97+OyXpC2brxKIJMiXWd82Le7386RKtOoum76tgmjasnlichoo0hoSDu
> DryE4yirPTIdwBwP6g88hpEjfFX7LUBkKPU2TixcUnjY3x2oD+gg/qKnswFvd7er
> +ae+NaUaCtnPs2Lj0ebWYJGXEPdqcvaFExJaUO5SWx6lyoIj55dU9o910KAkbOwH
> 2uk3WQzrL5/FuaVlv6Ilp/s7Ui4nQmCUmDvBJu1rdtoDu/F0WArK6AF8C28+zSY=
> =Ypgz
> -----END PGP SIGNATURE-----
> _______________________________________________
> poppler mailing list
> poppler at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/poppler
>
> .
>