<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr">On Fri, 27 Apr 2018, 04:48 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 27/04/2018 08:51, Joel Sherrill wrote:<br>
> <br>
>        // Create the set of desired symbols.<br>
>        SymbolsToAnalyze = new Coverage::DesiredSymbols();<br>
>     -  SymbolsToAnalyze->load( symbolsFile );<br>
> <br>
> <br>
> I am having trouble seeing what happened to the DesiredSymbols class. Is<br>
> it no longer needed?</blockquote></div></div><div dir="auto"><br></div><div dir="auto">It's still being used as before, the decision has just been split from did you provided the symbols file or are you using symbol_set.cfg file. DesiredSymbols is definitely still being used. I'm looking on my phone  so maybe missing something that would be clearer not on a phone.</div><div dir="auto"><br></div><div dir="auto">Just to update, I didn't get time to look at the rsb build yesterday so it'll have to be tonight. Sorry for the delay, my new job keeps me very busy during the week, long commutes too.</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"> Did</blockquote></div></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"> we change the class from which SymbolsToAnalyze<br>
> is derived but provide all the same needed interfaces?<br>
> <br>
> I'm having trouble seeing how the new class fits in. Help me.<br>
<br>
I am adding support to determine the build top from the executable path(s) and<br>
to verify the build path, arch prefix and BSP match for all executables passed<br>
in and then I will load the symbols from the libraries listed in the config file<br>
directly into the desired symbols class. It looks me like these extra classes<br>
are not needed.<br>
<br>
Chris<br>
<br>
_______________________________________________<br>
devel mailing list<br>
<a href="mailto:devel@rtems.org" rel="noreferrer noreferrer" target="_blank">devel@rtems.org</a><br>
<a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer noreferrer noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a></blockquote></div></div></div>