<div dir="ltr"><div><div><div><div><div><div>Hi Frank,<br></div>It's really a matter of personal style.  There's a multitude of different ways that people format multi-line function calls so no one way is right or wrong.  For instance:<br><div style="margin-left:40px"><br><span style="font-family:monospace,monospace">foo(</span><br><span style="font-family:monospace,monospace"></span><span style="font-family:monospace,monospace">  bar1</span><br><span style="font-family:monospace,monospace"></span><span style="font-family:monospace,monospace">  bar2)</span><br><span style="font-family:monospace,monospace"></span><br><span style="font-family:monospace,monospace"></span><span style="font-family:monospace,monospace">foo(</span><br><span style="font-family:monospace,monospace"></span><span style="font-family:monospace,monospace">  bar1</span><br><span style="font-family:monospace,monospace">  bar2</span><br><span style="font-family:monospace,monospace">  )</span><br><span style="font-family:monospace,monospace"></span><br><span style="font-family:monospace,monospace">foo(</span><br><span style="font-family:monospace,monospace">  bar1</span><br><span style="font-family:monospace,monospace">  bar2</span><br><span style="font-family:monospace,monospace">)<br><br></span><span style="font-family:monospace,monospace">foo(bar1</span><br></div><div style="margin-left:40px"><span style="font-family:monospace,monospace">    bar2)<br></span><span style="font-family:monospace,monospace"><span style="font-family:monospace,monospace"><br></span></span><span style="font-family:monospace,monospace"><span style="font-family:monospace,monospace">foo(bar1</span></span><span style="font-family:monospace,monospace"><br>    bar2</span><span style="font-family:monospace,monospace"></span><span style="font-family:monospace,monospace"><br>)</span><span style="font-family:monospace,monospace"><span style="font-family:monospace,monospace"><br><br>foo(bar1</span></span><span style="font-family:monospace,monospace"><br>    bar2</span><span style="font-family:monospace,monospace"></span><span style="font-family:monospace,monospace"><br>    )</span><span style="font-family:monospace,monospace"></span><br></div><br>All of which have their justifications.<br></div></div></div></div></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr">----------<br>Chuck Atkins<br>Staff R&D Engineer, Scientific Computing<br>Kitware, Inc.<br><br></div></div></div></div></div></div></div>
<br><div class="gmail_quote">On Fri, Jun 2, 2017 at 1:57 AM, Frank Roland <span dir="ltr"><<a href="mailto:theldoria@hotmail.com" target="_blank">theldoria@hotmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">




<div dir="ltr">
<div id="m_-4550679430207729507divtagdefaultwrapper" style="font-size:12pt;color:#000000;font-family:Calibri,Arial,Helvetica,sans-serif" dir="ltr">
<p>There is probably a faulty indentation implemented in cmake-mode for emacs regarding closing parens.</p>
<p>If you have the following source code:</p>
<p></p>
<div>
<div>
<div>foo(<br>
   bar<br>
</div>
<div>)<br>
</div>
</div>
<br>
</div>
<p></p>
<p><br>
</p>
<p>I indents the closing parenwould expect it to be indented like this:</p>
<p></p>
<div>foo(<br>
   bar<br>
   )<br>
</div>
<p></p>
<p><br>
</p>
<p>I would expect the single closing paren to be not indendet.</p>
<p><br>
</p>
<p>Can anybody confirm that this is an issue worth reporting?</p>
<p><br>
</p>
<p>I could also create a pull request for a fix if there is interest in doing so.<br>
</p>
<p><br>
</p>
</div>
</div>

<br>--<br>
<br>
Powered by <a href="http://www.kitware.com" rel="noreferrer" target="_blank">www.kitware.com</a><br>
<br>
Please keep messages on-topic and check the CMake FAQ at: <a href="http://www.cmake.org/Wiki/CMake_FAQ" rel="noreferrer" target="_blank">http://www.cmake.org/Wiki/<wbr>CMake_FAQ</a><br>
<br>
Kitware offers various services to support the CMake community. For more information on each offering, please visit:<br>
<br>
CMake Support: <a href="http://cmake.org/cmake/help/support.html" rel="noreferrer" target="_blank">http://cmake.org/cmake/help/<wbr>support.html</a><br>
CMake Consulting: <a href="http://cmake.org/cmake/help/consulting.html" rel="noreferrer" target="_blank">http://cmake.org/cmake/help/<wbr>consulting.html</a><br>
CMake Training Courses: <a href="http://cmake.org/cmake/help/training.html" rel="noreferrer" target="_blank">http://cmake.org/cmake/help/<wbr>training.html</a><br>
<br>
Visit other Kitware open-source projects at <a href="http://www.kitware.com/opensource/opensource.html" rel="noreferrer" target="_blank">http://www.kitware.com/<wbr>opensource/opensource.html</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="http://public.kitware.com/mailman/listinfo/cmake" rel="noreferrer" target="_blank">http://public.kitware.com/<wbr>mailman/listinfo/cmake</a><br></blockquote></div><br></div></div>