[PATCH] Write event stream in file

Sebastian Huber sebastian.huber at embedded-brains.de
Tue Jun 25 09:02:47 UTC 2019



On 25/06/2019 10:49, Ravindra Kumar Meena wrote:
>      >     [242/254] Compiling misc/record/record-main.c
>      >     ../misc/record/record-main.c: In function ‘print_item’:
>      >     ../misc/record/record-main.c:152:14: warning: variable
>     ‘seconds’ set
>      >     but
>      >     not used [-Wunused-but-set-variable]
>      >            uint32_t seconds;
>      >                     ^~~~~~~
>      >     ../misc/record/record-main.c: In function ‘handler’:
>      >     ../misc/record/record-main.c:161:14: warning: ‘ns’ may be used
>      >     uninitialized in this function [-Wmaybe-uninitialized]
>      >          ctf_item.ns=ns;
>      >          ~~~~~~~~~~~^~~
>      >     ../misc/record/record-main.c:149:12: note: ‘ns’ was declared here
>      >          uint32_t ns;
>      >                   ^~
>      >
>      > Okay. I also got these warnings after reconfiguring again. This
>     can be
>      > resolved if we remove the if block. I think there is no need for
>     it now.
> 
>     No, sorry, please fix these warnings immediately. Warnings about
>     uninitialized variables are very likely a hard error.
> 
> Okay. I have resolved the issue.
> https://github.com/rmeena840/rtems-tools/commit/23aeaa0251e89822ac24108f3d0c34af36b610fb
> 
> Have a look.

Why do you still have the "ns" variable and this if block? Why can't you 
simply do this:

ctf_item.ns = item->ns;

?

-- 
Sebastian Huber, embedded brains GmbH

Address : Dornierstr. 4, D-82178 Puchheim, Germany
Phone   : +49 89 189 47 41-16
Fax     : +49 89 189 47 41-09
E-Mail  : sebastian.huber at embedded-brains.de
PGP     : Public key available on request.

Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG.



More information about the devel mailing list