<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Oct 14, 2020, 6:03 PM Chris Johns <<a href="mailto:chrisj@rtems.org">chrisj@rtems.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 15/10/20 5:04 am, Joel Sherrill wrote:<br>
> On Wed, Oct 14, 2020 at 12:11 PM Sebastian Huber<br>
> <<a href="mailto:sebastian.huber@embedded-brains.de" target="_blank" rel="noreferrer">sebastian.huber@embedded-brains.de</a> <mailto:<a href="mailto:sebastian.huber@embedded-brains.de" target="_blank" rel="noreferrer">sebastian.huber@embedded-brains.de</a>>><br>
> wrote:<br>
>     Hello,<br>
> <br>
>     I tried to write a configuration file for Uncrustify which is close to<br>
>     the RTEMS coding style. This tool has really a lot of options. The<br>
>     configuration file has more than 3000 lines. Apparently this is not<br>
>     enough. I was not able to get function parameters properly aligned. For<br>
>     example:<br>
> <br>
>     -  Thread_queue_Context  queue_context;<br>
>     -  Thread_Control       *the_thread;<br>
>     +  Thread_queue_Context queue_context;<br>
>     +  Thread_Control      *the_thread;<br>
> <br>
>     It seems Uncrustify interprets the '*' as some sort of white space.<br>
>     Attached is my current configuration file. You can test it for example<br>
>     with cpukit/score/src/threadqenqueue.c.<br>
> <br>
> I'm not a huge fan of moving the * to after the type rather than against<br>
> the name but if we accepted that change to the style, would the parameter<br>
> names line up?<br>
<br>
I understand the history with multiple variable declarations per line but to me<br>
the type with the modifier grouped together reads better. In C++ I prefer that<br>
style because there are more type modifiers such as reference and the reference<br>
and address-of is the same character. For example a randomly selected API ...<br>
<br>
<a href="https://en.cppreference.com/w/cpp/container/vector/vector" rel="noreferrer noreferrer" target="_blank">https://en.cppreference.com/w/cpp/container/vector/vector</a><br>
<br>
... and<br>
<br>
 vector& operator=( const vector& other );<br>
<br>
I had never noticed the C style until I used C++ heavily many years ago.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">I don't care as much about the placement as if this let's uncrustify align the parameter names. </div><div dir="auto"><br></div><div dir="auto">This is definitely in the category of something I'd comprise ob if the tool gives us an acceptable pattern overall.</div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Chris<br>
</blockquote></div></div></div>