<div dir="auto">Coming in late but there isn't there a helper method to turn object names into strings from either 32 bit values or strings? Shouldn't that be used?</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Aug 9, 2019, 8:17 AM Sebastian Huber <<a href="mailto:sebastian.huber@embedded-brains.de">sebastian.huber@embedded-brains.de</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 09/08/2019 15:06, Ravindra Kumar Meena wrote:<br>
> rtems-main.c cleanup:<br>
> <a href="https://github.com/rmeena840/rtems-tools/commit/61d5dc45e43f0998ad9305d565926090215a5bdc" rel="noreferrer noreferrer" target="_blank">https://github.com/rmeena840/rtems-tools/commit/61d5dc45e43f0998ad9305d565926090215a5bdc</a><br>
> <br>
> Switch_event per CPU added:<br>
> <a href="https://github.com/rmeena840/rtems-tools/commit/8ffe8bd2adea27772a9ab0e578539dd99fa0174b" rel="noreferrer noreferrer" target="_blank">https://github.com/rmeena840/rtems-tools/commit/8ffe8bd2adea27772a9ab0e578539dd99fa0174b</a><br>
> <br>
> Have a look.<br>
<br>
Ok, good. Please always check if the babeltrace and Trace Compass give <br>
the right results after the changes.<br>
<br>
If you use cctx->switch_event[ item->cpu ] and similar a couple of times <br>
in a function, then please assign it to a local pointer and use it, e.g.<br>
<br>
switch_event *se = &cctx->switch_event[ item->cpu ];<br>
<br>
Please check all structure names and make the similar to the names used <br>
in the metadata, e.g. swich_event -> event_sched_switch;<br>
<br>
Please check the style of the if again, it should be:<br>
<br>
if ( condition ) {<br>
  ...<br>
} else {<br>
  ...<br>
}<br>
<br>
Check the white space and position of the { }.<br>
<br>
When you have a function like get_api_of_id() you return the api, so the <br>
variable should be named "api":<br>
<br>
   size_t api_id = get_api_of_id( cctx->thread_id_name[ item->cpu <br>
].thread_id );<br>
<br>
   size_t api = get_api_of_id( cctx->thread_id_name[ item->cpu <br>
].thread_id );<br>
<br>
Declare all variables at the top of the function, e.g.<br>
<br>
size_t api;<br>
...<br>
api = get_api_of_id( cctx->thread_id_name[ item->cpu ].thread_id );<br>
<br>
-- <br>
Sebastian Huber, embedded brains GmbH<br>
<br>
Address : Dornierstr. 4, D-82178 Puchheim, Germany<br>
Phone   : +49 89 189 47 41-16<br>
Fax     : +49 89 189 47 41-09<br>
E-Mail  : <a href="mailto:sebastian.huber@embedded-brains.de" target="_blank" rel="noreferrer">sebastian.huber@embedded-brains.de</a><br>
PGP     : Public key available on request.<br>
<br>
Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG.<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></blockquote></div>