<html><head><meta http-equiv="Content-Type" content="text/html charset=iso-8859-1"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">Hi Bill, David,<div class="">maybe Uncrustify can be of some help on managing the indentation braces check. It makes use of a configuration file with which you can specify which are the rules you want to apply to the source file.</div><div class=""><br class=""></div><div class=""><a href="http://uncrustify.sourceforge.net" class="">http://uncrustify.sourceforge.net</a></div><div class=""><br class=""></div><div class="">Best Regards</div><div class="">
<div style="color: rgb(0, 0, 0); font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><span class="Apple-style-span" style="border-collapse: separate; font-variant-ligatures: normal; font-variant-position: normal; font-variant-numeric: normal; font-variant-alternates: normal; font-variant-east-asian: normal; line-height: normal; border-spacing: 0px;"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; border-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-stroke-width: 0px;"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><span class="Apple-style-span" style="border-collapse: separate; font-variant: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; border-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-stroke-width: 0px;"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><span class="Apple-style-span" style="font-variant: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-stroke-width: 0px;"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><span class="Apple-style-span" style="font-variant: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-stroke-width: 0px;"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div class=""><div style="font-weight: normal; font-style: normal; border-spacing: 0px; border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; border-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-stroke-width: 0px;"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; border-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-stroke-width: 0px;"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; border-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-stroke-width: 0px;"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; border-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-stroke-width: 0px;"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; border-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-stroke-width: 0px;"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; border-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-stroke-width: 0px;"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; border-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-stroke-width: 0px;"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; border-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-stroke-width: 0px;"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div class=""><span class="Apple-style-span" style="font-size: 13px; border-collapse: collapse; color: rgb(80, 0, 80); font-family: arial, sans-serif;"><b class=""><div style="display: inline !important;" class=""><span style="font-weight: normal;" class="">---</span></div></b></span></div></div></span></div></span></div></span></div></span></div></span></div></span></div></span></div></span></div><span style="font-weight: normal; font-style: normal; border-spacing: 0px; font-family: arial, sans-serif; font-size: 13px; border-collapse: collapse; color: rgb(80, 0, 80);" class=""><b class="">.: Paolo Quadrani</b></span><div class=""><font class="Apple-style-span" color="#500050" face="arial, sans-serif" style="font-family: Helvetica; font-size: medium; font-weight: normal; font-style: normal; color: rgb(0, 0, 0); border-spacing: 0px;"><span class="Apple-style-span" style="border-collapse: collapse; font-size: 13px;"><b class=""></b></span></font><div style="font-weight: normal; font-style: normal; border-spacing: 0px; border-collapse: collapse; color: rgb(80, 0, 80); font-family: arial, sans-serif; font-size: 13px;" class=""><br class=""></div><div style="font-weight: normal; font-style: normal; border-spacing: 0px; border-collapse: collapse; color: rgb(80, 0, 80); font-family: arial, sans-serif; font-size: 13px;" class=""><font color="#FF6600" class="">::</font><font color="#333333" class=""> [<a href="http://www.neralab.it/" class="">nEraLab</a>] - [<a href="http://it.linkedin.com/in/paoloquadrani" target="_blank" class="">LinkedIN</a>] - [<a href="http://twitter.com/quadrani/" target="_blank" class="">Twitter</a>] - [<a href="http://www.slideshare.net/quadrani" target="_blank" class="">SlideShare</a>]</font></div><div style="font-weight: normal; font-style: normal; border-spacing: 0px; border-collapse: collapse; color: rgb(80, 0, 80); font-family: arial, sans-serif; font-size: 13px;" class=""><font color="#000000" face="arial" class=""><span style="border-collapse: separate; font-size: small;" class=""><font color="#500050" face="arial, sans-serif" class=""><span style="border-collapse: collapse;" class=""><br class=""></span></font></span></font></div><div class=""><div class=""><span class="Apple-style-span" style="border-spacing: 2px;">"</span><span class="Apple-style-span" style="border-spacing: 2px;"><span class="Apple-style-span" style="background-color: transparent;">It always seems impossible until it's done. (N. Mandela)</span></span><span class="Apple-style-span" style="border-spacing: 2px;">"</span></div></div></div></div></div></span></div></span></div></span></div></span></div></span></div>
</div>
<br class=""><div><blockquote type="cite" class=""><div class="">On 04 lug 2016, at 21:05, Sean McBride <<a href="mailto:sean@rogue-research.com" class="">sean@rogue-research.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div class="">On Sat, 2 Jul 2016 12:53:49 -0400, Bill Lorensen said:<br class=""><br class=""><blockquote type="cite" class="">vtk_reindent_code.py has trouble with for loops after if blocks that<br class="">do not have braces.<br class=""><br class="">See the attached file, around lines 3269 and 3290.<br class=""><br class="">For example, Original:<br class="">int vtkReebGraph::Implementation::GetNumberOfNodes()<br class="">{<br class=""> if(!this->NodeNumber)<br class=""> for(vtkIdType nodeId = 1; nodeId < this->MainNodeTable.Size; nodeId++)<br class=""> {<br class=""> // check if node is cleared<br class=""> if(!(this->GetNode(nodeId)->ArcUpId == ((int)-2)))<br class=""> this->NodeNumber++;<br class=""> }<br class=""></blockquote><br class="">clang-tidy can find ifs and loops with missing braces and even add them for you:<br class=""><<a href="http://clang.llvm.org/extra/clang-tidy/checks/readability-braces-around-statements.html" class="">http://clang.llvm.org/extra/clang-tidy/checks/readability-braces-around-statements.html</a>><br class=""><br class="">After this re-indentation change lands, I'll give it a try...<br class=""><br class="">Cheers,<br class=""><br class="">-- <br class="">____________________________________________________________<br class="">Sean McBride, B. Eng <a href="mailto:sean@rogue-research.com" class="">sean@rogue-research.com</a><br class="">Rogue Research <a href="http://www.rogue-research.com" class="">www.rogue-research.com</a> <br class="">Mac Software Developer Montréal, Québec, Canada<br class=""><br class=""><br class="">_______________________________________________<br class="">Powered by <a href="http://www.kitware.com" class="">www.kitware.com</a><br class=""><br class="">Visit other Kitware open-source projects at <a href="http://www.kitware.com/opensource/opensource.html" class="">http://www.kitware.com/opensource/opensource.html</a><br class=""><br class="">Search the list archives at: <a href="http://markmail.org/search/?q=vtk-developers" class="">http://markmail.org/search/?q=vtk-developers</a><br class=""><br class="">Follow this link to subscribe/unsubscribe:<br class=""><a href="http://public.kitware.com/mailman/listinfo/vtk-developers" class="">http://public.kitware.com/mailman/listinfo/vtk-developers</a><br class=""><br class=""></div></div></blockquote></div><br class=""></body></html>