On 4/6/2012 12:11 PM, Rolf Eike Beer wrote:
Attached is what I currently have.

Neat.  I like the change to "CheckCXX11Features" instead of "FindCXX11".

I'd prefer that the result variable names start in "CXX11_", so

  HAS_CXX11_STATIC_ASSERT

would be

  CXX11_HAS_STATIC_ASSERT

Please draft documentation of the CheckCXX11Features module.
It will be easier to review the implementation if I know
exactly what it is supposed to do ;)

Thanks,
-Brad
--

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