On 10/30/2013 10:30 AM, Stephen Kelly wrote:
> Ok. I've updated my topic to remove the policy logic.

Thanks.  Some comments:

+          onlyPlain = false;
+          break;

Can't that just be "return false" now?

Also please add the corresponding test case and base the topic
on v2.8.12~2 so we can merge it for 2.8.12.1 after it passes
in 'next'.

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