[PATCH] record-main-lttng.cc: Change size of empty array

Joel Sherrill joel at rtems.org
Fri Oct 1 14:28:24 UTC 2021


The short message is misleading. It isn't an empty array -- it is an
emptyName string. :)

I'm ok with the patch but I reviewed it before submission so another
ACK from someone else is appreciated.

On Thu, Sep 30, 2021 at 3:17 PM Ryan Long <ryan.long at oarcorp.com> wrote:
>
> Change size of kEmptyThreadName from THREAD_API_COUNT to
> THREAD_NAME_SIZE.
>
> Closes #4519
> ---
>  trace/record/record-main-lttng.cc | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/trace/record/record-main-lttng.cc b/trace/record/record-main-lttng.cc
> index 7cfa48c..faa762d 100644
> --- a/trace/record/record-main-lttng.cc
> +++ b/trace/record/record-main-lttng.cc
> @@ -59,7 +59,7 @@
>  #define BITS_PER_CHAR 8
>  #define COMPACT_HEADER_ID 31
>
> -static const uint8_t kEmptyThreadName[THREAD_API_COUNT] = "";
> +static const uint8_t kEmptyThreadName[THREAD_NAME_SIZE] = "";
>
>  static const uint8_t kUUID[] = {0x6a, 0x77, 0x15, 0xd0, 0xb5, 0x02, 0x4c, 0x65,
>                                  0x86, 0x78, 0x67, 0x77, 0xac, 0x7f, 0x75, 0x5a};
> --
> 1.8.3.1
>
> _______________________________________________
> devel mailing list
> devel at rtems.org
> http://lists.rtems.org/mailman/listinfo/devel


More information about the devel mailing list