On Fri, Aug 10, 2012 at 10:57 AM, Brad King <[email protected]> wrote: > On 08/10/2012 10:52 AM, Stephen Kelly wrote: >> Stephen Kelly wrote: >> >>> Brad King wrote: >>> >>>> Hi Folks, >>>> >>>> With 2.8.9 out we'd like to perform two sweeping style change commits: >>>> >>>> (1) Remove all remnants of trailing whitespace >>>> (2) Convert all CMake language code to use lower-case commands >>>> >> >> Do you also have a script to clear the contents of endforeach(), endif(), >> endmacro(), endfunction()? Or do you not want to do that? > > That's a bit more risky because sometimes they span multiple lines. > I'd rather not tackle it now. > > -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
But after the sweeping changes, I would certainly welcome clean up commits that clear the end*() arguments. "Clean up" topics are always welcome. We won't merge them in during the middle of "release candidates" -- but we'll do it immediately after a release for sure. Thx, D -- 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
