[poppler] Moving from qsort to std::sort

Leonard Rosenthol lrosenth at adobe.com
Fri Aug 27 05:30:09 PDT 2010


I think replacing the custom container classes with the std:: containers will be a big win across the board - performance, maintenance, stability, etc.  The std:: classes have had a LOT of folks work on them and they've been deployed in a LOT more solutions.  Use them.

While getting rid of goostring with std::string might also be appealing, I think that's a larger issues that wouldn't benefit as much as the containers.

Leonard

-----Original Message-----
From: poppler-bounces+leonardr=adobe.com at lists.freedesktop.org [mailto:poppler-bounces+leonardr=adobe.com at lists.freedesktop.org] On Behalf Of Albert Astals Cid
Sent: Thursday, August 26, 2010 6:28 PM
To: poppler at lists.freedesktop.org
Subject: [poppler] Moving from qsort to std::sort

Yes, i know i've been a huge non std:: proponent in poppler for long time 
being a PITA for lots of people.

But these people where using lists or strings for which we have a "solution" 
that works for our use cases as well as the std ones.

Now qsort vs std::sort is a interesting story, i get up to 30% speedups 
(callgrind says) (in the function itself, obviously not in the total runtime) 
just exchanging qsort for std::sort.

So let's discuss if we want to allow std or not. And if we allow it, are we 
going all in? Or just allow some cases? But which?

Opinions?

Albert
_______________________________________________
poppler mailing list
poppler at lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/poppler


More information about the poppler mailing list