On 02/23/2015 12:55 PM, Brad King wrote: > On 02/21/2015 04:33 PM, Domen Vrankar wrote: >> Attached are patches with fixed patch 3. > > Thanks. I'll start with the SystemTools change which I've split > out into its own commit and pushed here: > > http://review.source.kitware.com/#/c/19304/
That one is now in KWSys 'master'. Next I've taken the two cleanup changes: http://review.source.kitware.com/19315 http://review.source.kitware.com/19316 For the other two changes, please refactor things to avoid use of function-local classes. They will likely not work on the older compilers KWSys supports. Also, please use the coding style of "this->Member" when referencing class members from method code. 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://public.kitware.com/mailman/listinfo/cmake-developers
