clang-tidy: add Chrome-inspired checks

[lttng-tools.git] / src / common / context.cpp

diff --git a/src/common/context.cpp b/src/common/context.cpp
index ab61a8133386b026bf27972a168030c74d9c401f..60e30ad298d9edc9a9bf7a519dc0bba32cf794d3 100644 (file)
--- a/src/common/context.cpp
+++ b/src/common/context.cpp
@@ -5,17 +5,18 @@
  *
  */
 
-#include "context.h"
+#include "context.hpp"
+
+#include <common/error.hpp>
+#include <common/macros.hpp>
+
 #include <stddef.h>
 #include <string.h>
-#include <common/error.h>
-#include <common/macros.h>
 
-int parse_application_context(const char *str, char **out_provider_name,
-               char **out_ctx_name)
+int parse_application_context(const char *str, char **out_provider_name, char **out_ctx_name)
 {
        const char app_ctx_prefix[] = "$app.";
-       char *provider_name = NULL, *ctx_name = NULL;
+       char *provider_name = nullptr, *ctx_name = nullptr;
        size_t i, len, colon_pos = 0, provider_name_len, ctx_name_len;
 
        if (!str || !out_provider_name || !out_ctx_name) {
@@ -46,22 +47,20 @@ int parse_application_context(const char *str, char **out_provider_name,
         * No colon found or no ctx name ("$app.provider:") or no provider name
         * given ("$app.:..."), which is invalid.
         */
-       if (!colon_pos || colon_pos == len ||
-                       colon_pos == sizeof(app_ctx_prefix)) {
+       if (!colon_pos || colon_pos == len || colon_pos == sizeof(app_ctx_prefix)) {
                goto not_found;
        }
 
        provider_name_len = colon_pos - sizeof(app_ctx_prefix) + 2;
-       provider_name = (char *) zmalloc(provider_name_len);
+       provider_name = calloc<char>(provider_name_len);
        if (!provider_name) {
                PERROR("malloc provider_name");
                goto not_found;
        }
-       strncpy(provider_name, str + sizeof(app_ctx_prefix) - 1,
-                       provider_name_len - 1);
+       strncpy(provider_name, str + sizeof(app_ctx_prefix) - 1, provider_name_len - 1);
 
        ctx_name_len = len - colon_pos;
-       ctx_name = (char *) zmalloc(ctx_name_len);
+       ctx_name = calloc<char>(ctx_name_len);
        if (!ctx_name) {
                PERROR("malloc ctx_name");
                goto not_found;
@@ -76,4 +75,3 @@ not_found:
        free(ctx_name);
        return -1;
 }
-