[PATCH] Write event stream in file

Sebastian Huber sebastian.huber at embedded-brains.de
Tue Jun 25 08:19:39 UTC 2019


On 25/06/2019 09:48, Ravindra Kumar Meena wrote:
>      >     Do you get compiler warnings with this version?
>      >
>      >
>      > No I didn't get any compiler warning or error.
> 
>     How did you configure the build? Did you somehow disable the warnings?
> 
> 
>     [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.

-- 
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