El Dilluns, 26 de novembre de 2012, a les 11:56:54, Thomas Freitag va escriure: > 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),
Mailman was down for while, but all the messages should have been delivered already now. Let's keep the discussion in the bug. Cheers, Albert > 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 [email protected] > >> 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 > > [email protected] > > http://lists.freedesktop.org/mailman/listinfo/poppler > > > > . > > _______________________________________________ > poppler mailing list > [email protected] > http://lists.freedesktop.org/mailman/listinfo/poppler _______________________________________________ poppler mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/poppler
