2012/3/1 Brad King <brad.k...@kitware.com>: > On 3/1/2012 10:47 AM, Eric Noulard wrote: >> >> Last time I tried to enable KWStyle hooks following this: >> http://www.cmake.org/Wiki/Git/Hooks#Setup > > > That page is generic for many of Kitware's projects and > is not specific to CMake. The same hooks are also used > for ITK. For a while the kwstyle and uncrustify hooks > were added and used by ITK. Later they were moved over > to ITK proper and are now invoked through the generic > hooks' chaining feature. They no longer exist in the > generic hooks outright. I removed the discussion of > them from the wiki page. > > I'm not a big fan of automatic code formatting and > layout tools. There are almost always exceptions and > special cases. I'd rather cover that during code review.
Yes I agree with that but may be using them as pre-commit "warning" on changed files may be interesting. Nevertheless checking for no more than 79 column style certainly does not require such tool. -- Erk Le gouvernement représentatif n'est pas la démocratie -- http://www.le-message.org -- 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
