On 08/15/2013 07:40 AM, David Cole wrote:
> http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=b2ac3463b5ef5dfd3fc662c12aaaacdd177e5dc1
> 
> git says it's only in 'next' and 'nightly' so far:

Actually that one was rewritten and is in master:

 http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=58e52416

> The commit message explains the rationale. Not sure if Brad is aware 
> the warning can be triggered by parenthesized expressions in the if 
> command... Brad?

I was aware that a left paren after another token would warn but
did not think about the use case in if() expressions.  If I were
designing the language from scratch I probably wouldn't want to
allow this but it is probably fairly common now.  The purpose of
the warning was for other cases so it is okay to tolerate this.
Fixed and test added:

 http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=9040ec91

Thanks,
-Brad
--

Powered by www.kitware.com

Please keep messages on-topic and check the CMake FAQ at: 
http://www.cmake.org/Wiki/CMake_FAQ

Kitware offers various services to support the CMake community. For more 
information on each offering, please visit:

CMake Support: http://cmake.org/cmake/help/support.html
CMake Consulting: http://cmake.org/cmake/help/consulting.html
CMake Training Courses: http://cmake.org/cmake/help/training.html

Visit other Kitware open-source projects at 
http://www.kitware.com/opensource/opensource.html

Follow this link to subscribe/unsubscribe:
http://www.cmake.org/mailman/listinfo/cmake

Reply via email to