<div dir="ltr">Hi Matt,<div>Do KWStyle and <span style="font-size:12.8px">uncrustify play well with both enabled ?</span></div><div><span style="font-size:12.8px">I have </span> <span style="font-size:12.8px">uncrustify  version </span>0.63 and the prehooks still fail. I believe 0.63 is the latest uncrustify version: <a href="https://github.com/uncrustify/uncrustify/blob/master/CMakeLists.txt#L124">https://github.com/uncrustify/uncrustify/blob/master/CMakeLists.txt#L124</a></div><div><br></div><div>Thanks,</div><div>Somesh</div>







</div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Sep 12, 2016 at 5:12 PM, Matt McCormick <span dir="ltr"><<a href="mailto:matt.mccormick@kitware.com" target="_blank">matt.mccormick@kitware.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Somesh,<br>
<br>
Yes, Dzenan recently improved upstream uncrustify and its<br>
configuration in ITK so it works much better. A newer version of<br>
uncrustify is required as described here:<br>
<br>
  <a href="https://github.com/InsightSoftwareConsortium/ITK/blob/6ee10575445aa1c878797db4febf7d775a38f1f3/Utilities/DevelopmentSetupScripts/GitTips.sh#L66-L74" rel="noreferrer" target="_blank">https://github.com/<wbr>InsightSoftwareConsortium/ITK/<wbr>blob/<wbr>6ee10575445aa1c878797db4febf7d<wbr>775a38f1f3/Utilities/<wbr>DevelopmentSetupScripts/<wbr>GitTips.sh#L66-L74</a><br>
<br>
Thanks,<br>
Matt<br>
<div><div class="h5"><br>
On Mon, Sep 12, 2016 at 2:49 PM, Somesh <<a href="mailto:someshsandbox@gmail.com">someshsandbox@gmail.com</a>> wrote:<br>
> Hi,<br>
> I recently set up a ITK dev on my mac. While commiting a patch locally, I am<br>
> getting failure in precommit hooks. Specifically uncrustify is giving the<br>
> following errors. Do I have to install a specific version of<br>
> uncrustify/python ? I installed uncrustify using brew.<br>
><br>
> ./Utilities/Maintenance/<wbr>uncrustify_itk.cfg:99 Unknown symbol<br>
> 'indent_cs_delegate_brace'<br>
><br>
> ./Utilities/Maintenance/<wbr>uncrustify_itk.cfg:287 Unknown symbol<br>
> 'indent_token_after_brace'<br>
><br>
> ./Utilities/Maintenance/<wbr>uncrustify_itk.cfg:290 Unknown symbol<br>
> 'indent_cpp_lambda_body'<br>
><br>
> ./Utilities/Maintenance/<wbr>uncrustify_itk.cfg:424 Unknown symbol<br>
> 'sp_angle_paren_empty'<br>
><br>
> ./Utilities/Maintenance/<wbr>uncrustify_itk.cfg:533 Unknown symbol<br>
> 'sp_after_operator_sym_empty'<br>
><br>
> ./Utilities/Maintenance/<wbr>uncrustify_itk.cfg:570 Unknown symbol<br>
> 'sp_func_proto_paren_empty'<br>
><br>
> ./Utilities/Maintenance/<wbr>uncrustify_itk.cfg:576 Unknown symbol<br>
> 'sp_func_def_paren_empty'<br>
><br>
> ./Utilities/Maintenance/<wbr>uncrustify_itk.cfg:614 Unknown symbol<br>
> 'sp_func_class_paren_empty'<br>
><br>
> ./Utilities/Maintenance/<wbr>uncrustify_itk.cfg:644 Unknown symbol<br>
> 'sp_super_paren'<br>
><br>
> ./Utilities/Maintenance/<wbr>uncrustify_itk.cfg:647 Unknown symbol<br>
> 'sp_this_paren'<br>
><br>
> ./Utilities/Maintenance/<wbr>uncrustify_itk.cfg:851 Unknown symbol<br>
> 'sp_skip_vbrace_tokens'<br>
><br>
> ./Utilities/Maintenance/<wbr>uncrustify_itk.cfg:922 Unknown symbol<br>
> 'align_var_class_span'<br>
><br>
> ./Utilities/Maintenance/<wbr>uncrustify_itk.cfg:925 Unknown symbol<br>
> 'align_var_class_thresh'<br>
><br>
> ./Utilities/Maintenance/<wbr>uncrustify_itk.cfg:928 Unknown symbol<br>
> 'align_var_class_gap'<br>
><br>
> ./Utilities/Maintenance/<wbr>uncrustify_itk.cfg:1068 Unknown symbol<br>
> 'nl_oc_block_brace'<br>
><br>
> ./Utilities/Maintenance/<wbr>uncrustify_itk.cfg:1245 Unknown symbol<br>
> 'nl_enum_own_lines'<br>
><br>
> ./Utilities/Maintenance/<wbr>uncrustify_itk.cfg:1256 Unknown symbol<br>
> 'nl_func_class_scope'<br>
><br>
> ./Utilities/Maintenance/<wbr>uncrustify_itk.cfg:1284 Unknown symbol<br>
> 'nl_func_decl_start_multi_<wbr>line'<br>
><br>
> ./Utilities/Maintenance/<wbr>uncrustify_itk.cfg:1287 Unknown symbol<br>
> 'nl_func_def_start_multi_line'<br>
><br>
> ./Utilities/Maintenance/<wbr>uncrustify_itk.cfg:1296 Unknown symbol<br>
> 'nl_func_decl_args_multi_line'<br>
><br>
> ./Utilities/Maintenance/<wbr>uncrustify_itk.cfg:1299 Unknown symbol<br>
> 'nl_func_def_args_multi_line'<br>
><br>
> ./Utilities/Maintenance/<wbr>uncrustify_itk.cfg:1314 Unknown symbol<br>
> 'nl_func_decl_end_multi_line'<br>
><br>
> ./Utilities/Maintenance/<wbr>uncrustify_itk.cfg:1317 Unknown symbol<br>
> 'nl_func_def_end_multi_line'<br>
><br>
> ./Utilities/Maintenance/<wbr>uncrustify_itk.cfg:1326 Unknown symbol<br>
> 'nl_func_call_start_multi_<wbr>line'<br>
><br>
> ./Utilities/Maintenance/<wbr>uncrustify_itk.cfg:1329 Unknown symbol<br>
> 'nl_func_call_args_multi_line'<br>
><br>
> ./Utilities/Maintenance/<wbr>uncrustify_itk.cfg:1332 Unknown symbol<br>
> 'nl_func_call_end_multi_line'<br>
><br>
> ./Utilities/Maintenance/<wbr>uncrustify_itk.cfg:1388 Unknown symbol<br>
> 'nl_squeeze_ifdef_top_level'<br>
><br>
> ./Utilities/Maintenance/<wbr>uncrustify_itk.cfg:1461 Unknown symbol<br>
> 'nl_split_if_one_liner'<br>
><br>
> ./Utilities/Maintenance/<wbr>uncrustify_itk.cfg:1465 Unknown symbol<br>
> 'nl_split_for_one_liner'<br>
><br>
> ./Utilities/Maintenance/<wbr>uncrustify_itk.cfg:1469 Unknown symbol<br>
> 'nl_split_while_one_liner'<br>
><br>
> ./Utilities/Maintenance/<wbr>uncrustify_itk.cfg:1541 Unknown symbol<br>
> 'nl_after_func_class_proto'<br>
><br>
> ./Utilities/Maintenance/<wbr>uncrustify_itk.cfg:1544 Unknown symbol<br>
> 'nl_after_func_class_proto_<wbr>group'<br>
><br>
> ./Utilities/Maintenance/<wbr>uncrustify_itk.cfg:1662 Unknown symbol<br>
> 'mod_full_brace_if_chain_only'<br>
><br>
> ./Utilities/Maintenance/<wbr>uncrustify_itk.cfg:1781 Unknown symbol<br>
> 'cmt_multi_first_len_minimum'<br>
><br>
> ./Utilities/Maintenance/<wbr>uncrustify_itk.cfg:1812 Unknown symbol<br>
> 'cmt_insert_before_inlines'<br>
><br>
> ./Utilities/Maintenance/<wbr>uncrustify_itk.cfg:1817 Unknown symbol<br>
> 'cmt_insert_before_ctor_dtor'<br>
><br>
> ./Utilities/Maintenance/<wbr>uncrustify_itk.cfg:1878 Unknown symbol<br>
> 'use_options_overriding_for_<wbr>qt_macros'<br>
><br>
> ./Utilities/Maintenance/<wbr>uncrustify_itk.cfg:1885 Unknown symbol<br>
> 'warn_level_tabs_found_in_<wbr>verbatim_string_literals'<br>
><br>
><br>
> Thanks,<br>
> Somesh<br>
><br>
><br>
</div></div>> ______________________________<wbr>_________________<br>
> Powered by <a href="http://www.kitware.com" rel="noreferrer" target="_blank">www.kitware.com</a><br>
><br>
> Visit other Kitware open-source projects at<br>
> <a href="http://www.kitware.com/opensource/opensource.html" rel="noreferrer" target="_blank">http://www.kitware.com/<wbr>opensource/opensource.html</a><br>
><br>
> Kitware offers ITK Training Courses, for more information visit:<br>
> <a href="http://kitware.com/products/protraining.php" rel="noreferrer" target="_blank">http://kitware.com/products/<wbr>protraining.php</a><br>
><br>
> Please keep messages on-topic and check the ITK FAQ at:<br>
> <a href="http://www.itk.org/Wiki/ITK_FAQ" rel="noreferrer" target="_blank">http://www.itk.org/Wiki/ITK_<wbr>FAQ</a><br>
><br>
> Follow this link to subscribe/unsubscribe:<br>
> <a href="http://public.kitware.com/mailman/listinfo/insight-developers" rel="noreferrer" target="_blank">http://public.kitware.com/<wbr>mailman/listinfo/insight-<wbr>developers</a><br>
><br>
</blockquote></div><br></div>