<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr">On Wed, May 23, 2018, 9:59 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 24/5/18 12:07 pm, Joel Sherrill wrote:<br>
> ---<br>
>  tester/covoar/ObjdumpProcessor.cc | 22 ++++++++++++++++++++++<br>
>  1 file changed, 22 insertions(+)<br>
> <br>
> diff --git a/tester/covoar/ObjdumpProcessor.cc b/tester/covoar/ObjdumpProcessor.cc<br>
> index d41906c..183722b 100644<br>
> --- a/tester/covoar/ObjdumpProcessor.cc<br>
> +++ b/tester/covoar/ObjdumpProcessor.cc<br>
> @@ -384,6 +384,28 @@ namespace Coverage {<br>
>          break;<br>
>        }<br>
>  <br>
> +      // This string indicates a block of padding at the end of a method<br>
> +      if (!strncmp(line.c_str(), "\t...", 4)) {<br>
<br>
You have an std::string so why no use C++ rather than resorting to C?<br>
<br>
For example <a href="http://en.cppreference.com/w/cpp/string/basic_string/compare" rel="noreferrer noreferrer" target="_blank">http://en.cppreference.com/w/cpp/string/basic_string/compare</a>.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">I was in a hurry and had trouble getting this logic to trip at all. I still fix this. It does need to be like strncmp.</div><div dir="auto"><br></div><div dir="auto">The +4 is awful though</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>
_______________________________________________<br>
devel mailing list<br>
<a href="mailto:devel@rtems.org" target="_blank" rel="noreferrer">devel@rtems.org</a><br>
<a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
</blockquote></div></div></div>