Nicolas Desprès wrote: > It was fastest because it was not doing the right thing. I tried to patch > it properly and the benchmark are the same whether we use the default > comparison functor or mine. > > So I think you can merge it like that. I have pushed a new version without > the comment. >
I still haven't tried it, but there are still style issues. * Don't put an else after a return * Wrap single line blocks in {} Thanks, Steve. -- Powered by www.kitware.com Visit other Kitware open-source projects at http://www.kitware.com/opensource/opensource.html Please keep messages on-topic and check the CMake FAQ at: http://www.cmake.org/Wiki/CMake_FAQ Follow this link to subscribe/unsubscribe: http://public.kitware.com/cgi-bin/mailman/listinfo/cmake-developers