<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr">On Wed, May 23, 2018, 10:01 PM Chris Johns <<a href="mailto:chrisj@rtems.org" target="_blank" rel="noreferrer">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/Target_sparc.cc | 4 ++++<br>
>  1 file changed, 4 insertions(+)<br>
> <br>
> diff --git a/tester/covoar/Target_sparc.cc b/tester/covoar/Target_sparc.cc<br>
> index 0e4ab2e..deeafef 100644<br>
> --- a/tester/covoar/Target_sparc.cc<br>
> +++ b/tester/covoar/Target_sparc.cc<br>
> @@ -64,6 +64,10 @@ namespace Target {<br>
>        size = 4;<br>
>        return true;<br>
>      }<br>
> +    if (!strcmp( &line[strlen(line)-6], "unimpl")) {<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">It's matching the last N characters on the line. It should be backing off one extra for a new line.</div><div dir="auto"><br></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>
I have not looked at the code around this for C++.<br>
<br>
Also these calls should be "::strcmp()".<br></blockquote></div></div><div dir="auto"><br></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"></blockquote></div></div><div dir="auto">Ok. The same pattern is every Target*.cc</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>
> +      size = 4;<br>
> +      return true;<br>
> +    }<br>
>  <br>
>      if (!strcmp( &line[strlen(line)-7], "unknown")) {<br>
>        size = 4; <br>
> <br>
</blockquote></div></div></div>