<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jan 13, 2017 at 4:47 AM, Sebastian Huber <span dir="ltr"><<a href="mailto:sebastian.huber@embedded-brains.de" target="_blank">sebastian.huber@embedded-<wbr>brains.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span>On 13/01/17 09:48, Sebastian Huber wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
On 12/01/17 22:44, Joel Sherrill wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
I repeat that I would like to purge old tests that result in conditionals in<br>
source code we no longer need. So any HAVE_XXX that are a side<br>
effect of transitioning newlib additions should be removed.<br>
</blockquote>
<br>
I removed three tests today which I added in the last two years or so. <br>
</blockquote>
<br></span>
I removed a couple of more obsolete configure stuff. I am done for now.<div class="gmail-m_-3395693159454090333HOEnZb"><div class="gmail-m_-3395693159454090333h5"><br></div></div></blockquote><div><br></div><div>Thanks. It is a continuous battle of creep and push back. </div><div><br></div><div>Without looking, I am sure you are right that <span style="font-size:12.8px">__RTEMS_DO_NOT_</span><wbr style="font-size:12.8px"><span style="font-size:12.8px">INLINE_THREAD_ENABLE_DISPATCH_</span><wbr style="font-size:12.8px"><span style="font-size:12.8px">_</span></div><div><span style="font-size:12.8px">is no longer used but it did serve a purpose that may have been lost in all the</span></div><div><span style="font-size:12.8px">rework.</span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">When doing coverage testing, inlining a method with one of more conditionals</span></div><div><span style="font-size:12.8px">increases the cyclomatic complexity of the calling method and number of test</span></div><div><span style="font-size:12.8px">cases required to do full coverage of all branch paths. That flag eliminated</span></div><div><span style="font-size:12.8px">at least 250 paths to test in the generated code.</span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">With the rework, did anything get inlined lots of times that includes a </span></div><div><span style="font-size:12.8px">conditional test? If so, then the calling method has hidden cyclomatic</span></div><div><span style="font-size:12.8px">complexity and requires increased test cases for full coverage.</span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">I am pretty sure this is written down as advice in the coverage area</span></div><div><span style="font-size:12.8px">but I have no idea how to turn it into good solid coding style/rules.</span></div><div><span style="font-size:12.8px">The best I can formulate is:</span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">Inlining a method requires that it be fully tested </span><span style="font-size:12.8px">in the context of each </span></div><div><span style="font-size:12.8px">place it is called. Inlining complex methods with conditionals is undesirable</span></div><div><span style="font-size:12.8px">from a coverage testing viewpoint. Try to inline only simple methods.</span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">--joel</span></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div class="gmail-m_-3395693159454090333HOEnZb"><div class="gmail-m_-3395693159454090333h5">
<br>
-- <br>
Sebastian Huber, embedded brains GmbH<br>
<br>
Address : Dornierstr. 4, D-82178 Puchheim, Germany<br>
Phone   : <a href="tel:%2B49%2089%20189%2047%2041-16" value="+4989189474116" target="_blank">+49 89 189 47 41-16</a><br>
Fax     : <a href="tel:%2B49%2089%20189%2047%2041-09" value="+4989189474109" target="_blank">+49 89 189 47 41-09</a><br>
E-Mail  : <a href="mailto:sebastian.huber@embedded-brains.de" target="_blank">sebastian.huber@embedded-brain<wbr>s.de</a><br>
PGP     : Public key available on request.<br>
<br>
Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG.<br>
<br>
</div></div></blockquote></div><br></div></div>