clang-tidy: add Chrome-inspired checks

[lttng-tools.git] / src / common / actions / stop-session.cpp

diff --git a/src/common/actions/stop-session.cpp b/src/common/actions/stop-session.cpp
index 1a01f39255d927616ca8f66301f6e35af9c5f14e..9db4a25fc7104cb083fa16836b13fda91d5baa87 100644 (file)
--- a/src/common/actions/stop-session.cpp
+++ b/src/common/actions/stop-session.cpp
@@ -172,9 +172,9 @@ ssize_t lttng_action_stop_session_create_from_payload(struct lttng_payload_view
        ssize_t consumed_len, ret;
        const struct lttng_action_stop_session_comm *comm;
        const char *session_name;
        ssize_t consumed_len, ret;
        const struct lttng_action_stop_session_comm *comm;
        const char *session_name;

-       struct lttng_action *action = NULL;
+       struct lttng_action *action = nullptr;

        enum lttng_action_status status;
        enum lttng_action_status status;

-       struct lttng_rate_policy *policy = NULL;
+       struct lttng_rate_policy *policy = nullptr;

 
        comm = (typeof(comm)) view->buffer.data;
        session_name = (const char *) &comm->data;
 
        comm = (typeof(comm)) view->buffer.data;
        session_name = (const char *) &comm->data;


@@ -219,7 +219,7 @@ ssize_t lttng_action_stop_session_create_from_payload(struct lttng_payload_view
        }
 
        *p_action = action;
        }
 
        *p_action = action;

-       action = NULL;
+       action = nullptr;

 
 end:
        lttng_rate_policy_destroy(policy);
 
 end:
        lttng_rate_policy_destroy(policy);


@@ -234,19 +234,19 @@ lttng_action_stop_session_mi_serialize(const struct lttng_action *action, struct
        int ret;
        enum lttng_error_code ret_code;
        enum lttng_action_status status;
        int ret;
        enum lttng_error_code ret_code;
        enum lttng_action_status status;

-       const char *session_name = NULL;
-       const struct lttng_rate_policy *policy = NULL;
+       const char *session_name = nullptr;
+       const struct lttng_rate_policy *policy = nullptr;

 
        LTTNG_ASSERT(action);
        LTTNG_ASSERT(IS_STOP_SESSION_ACTION(action));
 
        status = lttng_action_stop_session_get_session_name(action, &session_name);
        LTTNG_ASSERT(status == LTTNG_ACTION_STATUS_OK);
 
        LTTNG_ASSERT(action);
        LTTNG_ASSERT(IS_STOP_SESSION_ACTION(action));
 
        status = lttng_action_stop_session_get_session_name(action, &session_name);
        LTTNG_ASSERT(status == LTTNG_ACTION_STATUS_OK);

-       LTTNG_ASSERT(session_name != NULL);
+       LTTNG_ASSERT(session_name != nullptr);

 
        status = lttng_action_stop_session_get_rate_policy(action, &policy);
        LTTNG_ASSERT(status == LTTNG_ACTION_STATUS_OK);
 
        status = lttng_action_stop_session_get_rate_policy(action, &policy);
        LTTNG_ASSERT(status == LTTNG_ACTION_STATUS_OK);

-       LTTNG_ASSERT(policy != NULL);
+       LTTNG_ASSERT(policy != nullptr);

 
        /* Open action stop session. */
        ret = mi_lttng_writer_open_element(writer, mi_lttng_element_action_start_session);
 
        /* Open action stop session. */
        ret = mi_lttng_writer_open_element(writer, mi_lttng_element_action_start_session);


@@ -284,8 +284,8 @@ end:
 
 struct lttng_action *lttng_action_stop_session_create(void)
 {
 
 struct lttng_action *lttng_action_stop_session_create(void)
 {

-       struct lttng_action_stop_session *action_stop = NULL;
-       struct lttng_rate_policy *policy = NULL;
+       struct lttng_action_stop_session *action_stop = nullptr;
+       struct lttng_rate_policy *policy = nullptr;

        enum lttng_action_status status;
 
        /* Create a every N = 1 rate policy. */
        enum lttng_action_status status;
 
        /* Create a every N = 1 rate policy. */


@@ -312,7 +312,7 @@ struct lttng_action *lttng_action_stop_session_create(void)
        status = lttng_action_stop_session_set_rate_policy(&action_stop->parent, policy);
        if (status != LTTNG_ACTION_STATUS_OK) {
                lttng_action_destroy(&action_stop->parent);
        status = lttng_action_stop_session_set_rate_policy(&action_stop->parent, policy);
        if (status != LTTNG_ACTION_STATUS_OK) {
                lttng_action_destroy(&action_stop->parent);

-               action_stop = NULL;
+               action_stop = nullptr;

                goto end;
        }
 
                goto end;
        }
 


@@ -375,7 +375,7 @@ lttng_action_stop_session_set_rate_policy(struct lttng_action *action,
 {
        enum lttng_action_status status;
        struct lttng_action_stop_session *stop_session_action;
 {
        enum lttng_action_status status;
        struct lttng_action_stop_session *stop_session_action;

-       struct lttng_rate_policy *copy = NULL;
+       struct lttng_rate_policy *copy = nullptr;

 
        if (!action || !policy || !IS_STOP_SESSION_ACTION(action)) {
                status = LTTNG_ACTION_STATUS_INVALID;
 
        if (!action || !policy || !IS_STOP_SESSION_ACTION(action)) {
                status = LTTNG_ACTION_STATUS_INVALID;


@@ -394,7 +394,7 @@ lttng_action_stop_session_set_rate_policy(struct lttng_action *action,
 
        stop_session_action->policy = copy;
        status = LTTNG_ACTION_STATUS_OK;
 
        stop_session_action->policy = copy;
        status = LTTNG_ACTION_STATUS_OK;

-       copy = NULL;
+       copy = nullptr;

 
 end:
        lttng_rate_policy_destroy(copy);
 
 end:
        lttng_rate_policy_destroy(copy);