From 9cd8a08b64f236f580ef1519415be5cb8666f6e4 Mon Sep 17 00:00:00 2001
From: John Bytheway <jbytheway@gmail.com>
Date: Thu, 1 Aug 2019 21:32:59 -0400
Subject: [PATCH] clang-tidy readability-non-const-parameter (#32835)

* Enable readability-non-const-parameter

* Make a function parameter pointer-to-const
---
 .clang-tidy             | 1 -
 tests/behavior_test.cpp | 2 +-
 2 files changed, 1 insertion(+), 2 deletions(-)

diff --git a/.clang-tidy b/.clang-tidy
index 7826a927f9f53..86fb99d99de1c 100644
--- a/.clang-tidy
+++ b/.clang-tidy
@@ -71,7 +71,6 @@ readability-*,\
 -readability-isolate-declaration,\
 -readability-magic-numbers,\
 -readability-named-parameter,\
--readability-non-const-parameter,\
 -readability-container-size-empty,\
 -readability-redundant-control-flow,\
 -readability-redundant-declaration,\
diff --git a/tests/behavior_test.cpp b/tests/behavior_test.cpp
index 626de482ee11a..ae6c7bb4183dd 100644
--- a/tests/behavior_test.cpp
+++ b/tests/behavior_test.cpp
@@ -19,7 +19,7 @@ extern fallback_t default_fallback;
 extern sequential_until_done_t default_until_done;
 } // namespace behavior
 
-static behavior::node_t make_test_node( std::string goal, behavior::status_t *status )
+static behavior::node_t make_test_node( std::string goal, const behavior::status_t *status )
 {
     behavior::node_t node;
     if( !goal.empty() ) {