<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Jun 13, 2023, 4:43 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 14/6/2023 5:47 am, Joel Sherrill wrote:<br>
> <br>
> <br>
> On Tue, Jun 13, 2023 at 9:51 AM 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>
> <br>
>     On 13.06.23 00:04, Gedare Bloom wrote:<br>
>     >       "b _ARM_Exception_default\n"<br>
>     >       :<br>
>     > -    : [cpufsz] "i" (sizeof(CPU_Exception_frame)),<br>
>     > -      [cpuspoff] "i" (offsetof(CPU_Exception_frame, register_sp)),<br>
>     > -      [v7mlroff] "i" (offsetof(ARMV7M_Exception_frame, register_lr)),<br>
>     > -      [cpuvecoff] "J" (offsetof(CPU_Exception_frame, vector)),<br>
>     > -      [cpuvfpoff] "i" (ARM_EXCEPTION_FRAME_VFP_CONTEXT_OFFSET),<br>
>     > -      [cpacr] "i" (ARMV7M_CPACR),<br>
>     > -      [vfpsz] "i" (ARM_VFP_CONTEXT_SIZE)<br>
>     > +    : [cpufsz] "i"( sizeof( CPU_Exception_frame ) ),<br>
> <br>
>     If we place operators (e.g. &&, ||, ...) at the end of a broken line,<br>
>     then we should do this for the : as well.<br>
> <br>
> <br>
> OK.<br>
>  <br>
> <br>
> <br>
>     My current preference would be to format all non-third-party sources<br>
>     with a standard clang-format selection. I guess in the long run, this<br>
>     will be the easiest approach to maintain. If we use exotic options, then<br>
>     we may up ending as clang-format maintainers.<br>
> <br>
> <br>
> I think this is the thing we have to keep in mind and I even said I would<br>
> go along with compromises when we started. Get as close as you can<br>
> and we will have to accept that -- for now. We should definitely file tickets <br>
> with clang-format and ourselves to track things we don't like. If we get an <br>
> option in the future whether we or someone else implements it, we can <br>
> use it and reformat again. Those hopefully are not that invasive.<br>
<br>
Sounds like a plan.<br>
<br>
I am a little concerned about the version of clang-format I need as some<br>
machines I work on are no at current OS versions.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">I am finding myself experimenting with it for a customer's C++ project. So far, I have not matched the style. But I am now on Rocky 9 since I built it from source and needed cmake 3 at least. So you can't be on an old distribution and build from source.</div><div dir="auto"><br></div><div dir="auto">And Doxygen has and three different major versions between RHEL 7, 8, and 9. Using a configuration file from one on another seems to give you warnings about unknown options or obsolete options. </div><div dir="auto"><br></div><div dir="auto">Mixing new and old systems is a pain.</div><div dir="auto"><br></div><div dir="auto">--joel</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">
Chris<br>
</blockquote></div></div></div>