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

Reply via email to