<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Sep 10, 2020, 5:22 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 11/9/20 1:32 am, Joel Sherrill wrote:<br>
> I also think when doing this we should consider things that we do that<br>
> we have since learned safety standards don't like such as single statement<br>
> if's without braces. I think we should have braces now.<br>
<br>
Are there open tools available that check for these things available? Does this<br>
have to be linked to changing the format?<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">This should be an option. Requiring braces on all ifs isn't uncommon.</div><div dir="auto"><br></div><div dir="auto">I think the justification.for picking a format is being driven by having a tool to enforce it. I'd that's the sole factor, we can pick any old crappy format and meet that single requirement.</div><div dir="auto"><br></div><div dir="auto">I want to add the requirement that the format is not picked solely based on being something a tool already has a config file on. I am adding two requirements. </div><div dir="auto"><br></div><div dir="auto">(1) Clear identification of deltas from current style. If tools A and B miss by different amounts, I want us to.make a conscious decision on which to use.</div><div dir="auto"><br></div><div dir="auto">(2) Let's make sure we look at tool configuration options like always use.bracea that improve safety code reviews. </div><div dir="auto"><br></div><div dir="auto"> I don't mind some.changes -- I just.want them to be consciously evaluated.</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">
<br>
Chris<br>
</blockquote></div></div></div>